Skip to content

Conversation

javiereguiluz
Copy link
Member

No description provided.

@javiereguiluz javiereguiluz added this to the 4.0 milestone Nov 10, 2017
.. code-block:: terminal
$ bin/console config:dump-reference assetic
$ bin/console config:dump-reference WebProfilerBundle
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

php bin/console config:dump-reference WebProfilerBundle

$ bin/console config:dump-reference WebProfilerBundle
# you can also pass the bundle alias used in its config files:
# bin/console config:dump-reference web_profiler
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ php bin/console ...

.. code-block:: terminal
$ bin/console config:dump-reference AsseticBundle use_controller
$ bin/console config:dump-reference TwigBundle date
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

php bin/console

If the bundle included web assets, remove them from the ``public/bundles/``
directory (e.g. ``public/bundles/nelmioapidoc/`` for the NelmioApiDocBundle).

3. Remove Integration in other Bundles
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whoa, we shouldn't edit third party bundles to remove their integration? I know you don't mean that, but that title seems to indicate it (and I'm also not very sure what you do mean with this section).

As soon as bundle A depends on B, composer will not allow you to remove B so I don't think there is a problem here?

What we can say is that you have to remove the integration in your application. But that's rather obvious...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@weaverryan
Copy link
Member

In #8681 I've also proposed removing the remove.rst article entirely

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants