-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Don't use Flex shortcuts when installing packages #9742
Conversation
doctrine.rst
Outdated
@@ -23,8 +23,8 @@ code: | |||
|
|||
.. code-block:: terminal | |||
|
|||
composer require doctrine | |||
composer require maker --dev | |||
$ composer require doctrine/doctrine-bundle |
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.
By the way, why don't we use orm
here instead? With this step the required doctrine/orm
package would be missing leading to errors.
doctrine.rst
Outdated
@@ -479,7 +479,7 @@ for you automatically! First, install the bundle in case you don't have it: | |||
|
|||
.. code-block:: terminal | |||
|
|||
$ composer require annotations | |||
$ composer require doctrine/annotations |
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.
That's not the same, annotations
is an alias for the SensioFrameworkExtraBundle.
doctrine/registration_form.rst
Outdated
@@ -14,7 +14,7 @@ First, make sure you have all the dependencies you need installed: | |||
|
|||
.. code-block:: terminal | |||
|
|||
$ composer require doctrine form security validator | |||
$ composer require doctrine/doctrine-bundle symfony/form symfony/security-bundle symfony/validator |
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.
Same as above: use orm
instead?
page_creation.rst
Outdated
@@ -195,15 +195,15 @@ To use the web debug toolbar, just install it: | |||
|
|||
.. code-block:: terminal | |||
|
|||
$ composer require --dev profiler | |||
$ composer require --dev symfony/web-profiler-bundle |
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.
should be symfony/profiler-pack
profiler.rst
Outdated
@@ -12,7 +12,7 @@ install the profiler before using it: | |||
|
|||
.. code-block:: terminal | |||
|
|||
$ composer require profiler --dev | |||
$ composer require symfony/web-profiler-bundle --dev |
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.
symfony/profiler-pack
routing.rst
Outdated
@@ -24,7 +24,7 @@ First, install the annotations package: | |||
|
|||
.. code-block:: terminal | |||
|
|||
$ composer require annotations | |||
$ composer require doctrine/annotations |
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.
SensioFrameworkExtraBundle
security.rst
Outdated
@@ -136,7 +136,7 @@ You'll learn later how to deny access to certain URLs or controllers. | |||
|
|||
.. code-block:: terminal | |||
|
|||
$ composer require profiler --dev | |||
$ composer require symfony/web-profiler-bundle --dev |
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.
symfony/profiler-pack
serializer.rst
Outdated
@@ -123,7 +123,7 @@ To use annotations, first install the annotations package: | |||
|
|||
.. code-block:: terminal | |||
|
|||
$ composer require annotations | |||
$ composer require doctrine/annotations |
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.
SensioFrameworkExtraBundle
@xabbuh thanks for the review. I'm sorry I made so many mistakes. Hopefully they are fixed now! |
@@ -1,7 +1,7 @@ | |||
.. note:: | |||
|
|||
In order to use the annotation loader, you should have installed the | |||
``doctrine/annotations`` and ``doctrine/cache`` packages with Composer. | |||
``sensio/framework-extra-bundle`` and ``doctrine/cache`` packages with Composer. |
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 am not sure where we include this file, but is it needed in that context to have the SensioFrameworkExtraBundle or are these cases where doctrine/annotations
is actually fine?
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.
We include this file in:
- /components/routing.rst
- /components/serializer.rst
- /components/validator/resources.rst
In 2.7, 2.8 and 3.4 we showed this recommendation:
.. note::
In order to use the annotation loader, you should have installed the
``doctrine/annotations`` and ``doctrine/cache`` packages with Composer.
So let's keep it unchanged.
…eguiluz) This PR was squashed before being merged into the 4.0 branch (closes #9742). Discussion ---------- Don't use Flex shortcuts when installing packages This fixes #9693. I've left some shortcuts on purpose: * in `frontend/encore/installation.rst`: `composer require encore` because Webpack Encore is a bit special and the shortcut is the best way to install it. * in `quick_tour/*.rst` because it shows the features provided by Symfony and shortcuts is one of those features. Readers will learn about them and later they'll decide if they want to use them or not. * in `setup/flex.rst` because they are used on purpose to talk about Flex shortcuts. Commits ------- 2495590 Don't use Flex shortcuts when installing packages
This fixes #9693. I've left some shortcuts on purpose:
frontend/encore/installation.rst
:composer require encore
because Webpack Encore is a bit special and the shortcut is the best way to install it.quick_tour/*.rst
because it shows the features provided by Symfony and shortcuts is one of those features. Readers will learn about them and later they'll decide if they want to use them or not.setup/flex.rst
because they are used on purpose to talk about Flex shortcuts.