-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Simplified some Symfony installation instructions #4811
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simplified some Symfony installation instructions #4811
Conversation
@@ -84,19 +84,22 @@ to meet those requirements. | |||
distributing them. If you want to verify the integrity of any Symfony | |||
version, follow the steps `explained in this post`_. | |||
|
|||
Basing your Project on a Specific Symfony Version | |||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | |||
Basing your Project on a Specific Symfony Branch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can still consider 2.3 a version and not a branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 (and this should then also be reverted in the paragraph below)
@javiereguiluz Do you know when the phar will be updated? It would also be nice to make the phars available for each release on the Symfony Installer's GitHub release page. |
this is confusing me. So we recommended the Symfony directory structure. |
@timglabisch please open a dedicated issue for this message, because it is not related to changes done in this PR. It is a different topic |
Can you please fix the sphinx issues? Besides that, I believe this one is ready? |
Sorry for the delay! In a4451ce I've pushed a lot of new changes. Please review them. And regarding the merge of this PR, please wait a few days so we can release the new version of the installer (hopefully with support for the new |
|
||
Then, move the downloaded ``symfony.phar`` file to your projects directory and | ||
Then, move the downloaded ``symfony`` file to your projects directory and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there no way to install the command globally on Windows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here there are some instructions proposed by a Symfony Installer user (https://github.com/symfony/symfony-installer/issues/117) but I haven't tested them. Ideally we'd provide an executable installer as Composer does.
Now that the installer has been introduced officially, this PR is now ready to be merged. Thanks. |
Very nice! Should be ready to be merged now. |
…ereguiluz) This PR was merged into the 2.3 branch. Discussion ---------- Simplified some Symfony installation instructions | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | all | Fixed tickets | #4672 (partially) Before merging, the Symfony Installer must release its new version. Last time I checked, creating a new project based on a branch was ready in the GitHub repository, but not in the distributed installer. Commits ------- 004e0ad Moved the location of the labels c8b16cf Added labels for removed headings 11ac563 Fixed some bash comments a4451ce Updated the new Symfony installation instructions 4904d36 Simplified some Symfony installation instructions
Yay! Thanks Javier! |
Before merging, the Symfony Installer must release its new version. Last time I checked, creating a new project based on a branch was ready in the GitHub repository, but not in the distributed installer.