Rewritten from scratch the chapter about installing Symfony #4654
Conversation
javiereguiluz
commented
Dec 15, 2014
javiereguiluz
commented
| Q | A |
|---|---|
| Doc fix? | yes |
| New docs? | yes |
| Applies to | all |
| Fixed tickets | #4122, #4089 |
| The installer requires PHP 5.4 or higher. If you still use the legacy | ||
| PHP 5.3 version, you cannot use the Symfony Installer. In that case, read the next | ||
| :ref:`Creating Symfony Applications without the Installer <book-creating-applications-without-the-installer>` | ||
| section. |
There was a problem hiding this comment.
The sentence In that case, read the next Creating Symfony Applications without the Installer section looks weird IMO
to move symfony.phar to projects directory
| --------------------------------- | ||
| The installer requires PHP 5.4 or higher. If you still use the legacy | ||
| PHP 5.3 version, you cannot use the Symfony Installer. Read | ||
| :ref:`Creating Symfony Applications without the Installer <book-creating-applications-without-the-installer>` |
There was a problem hiding this comment.
I would change this sentence to this ("the" is missing, the ref label is not needed and "know" doesn't seem to be the right word to me):
Read the :ref:`book-creating-applications-without-the-installer` section to learn how to proceed.There was a problem hiding this comment.
Your proposal is much better. Thanks!
|
Looks great, Javier! 👍 |
|
Thank god for this - I will review shortly (I just ran out of time right now). I don't think that the plans I laid out in #4672 nullify this, I actually think this will be a major step forward towards that. |
…fony (javiereguiluz) This PR was merged into the 2.3 branch. Discussion ---------- Rewritten from scratch the chapter about installing Symfony | Q | A | ------------- | --- | Doc fix? | yes | New docs? | yes | Applies to | all | Fixed tickets | #4122, #4089 Commits ------- 7de83a3 Replaced a "note" by a "seealso" b45c338 Imrpoved the wording of the note about using using legacy PHP 5.3 version 40d7772 Reworded the reasons why you should use Composer-based installation 2e72138 Wrapped a line to follow doc standards 7e9cd04 Display the same version number on Linux and Windows to avoid confusions 12c2557 Add an explicit command to better explain that we recommend to move symfony.phar to projects directory 12eb76e Reworded a confusing phrase b665d86 Added a note about verifying Symfony digital signatures 8f202c6 Re-added a wrongly deleted link reference c291a73 Rewritten from scratch the chapter about installing Symfony
|
Thanks so much Javier! When I created my list for #4672, this was - by far - the ugliest and worst todo on that list. I think we can take another look at it once the new download page is there, but it's already many times better than before. Cheers! |
