Updated the main bundles article for Symfony 4 #8613
Conversation
| are used by your application (including the core Symfony bundles). | ||
| .. caution:: | ||
|
|
||
| In Symfony versions prior to 4.0, it was recommended to organize your own |
There was a problem hiding this comment.
I do not really agree with this sentence. At least, it is a bit confusing as we didn't recommend to use bundles, but there only was a single AppBundle. I think that's not really clear right now.
There was a problem hiding this comment.
We'll need more opinions here, but I thought this:
- Symfony 2: organize your own code in bundles (1 bundle per feature: UserBundle, ProuctBundle, ...)
- Symfony 3: organize your own code in bundles (just 1 bundle per app: AppBundle)
Saying: "in prior Sf versions we recommended you to organize your own code in bundles" is compatible with that (in my opinion).
There was a problem hiding this comment.
I agree with @javiereguiluz here. "organize in bundles" doesn't have to refer to multiple bundles
| are used by your application (including the core Symfony bundles). | ||
| .. caution:: | ||
|
|
||
| In Symfony versions prior to 4.0, it was recommended to organize your own |
There was a problem hiding this comment.
I agree with @javiereguiluz here. "organize in bundles" doesn't have to refer to multiple bundles
bundles.rst
Outdated
| In Symfony versions prior to 4.0, it was recommended to organize your own | ||
| application code using bundles. This is no longer recommended and bundles | ||
| should only be used as third-party plugins that add features in your | ||
| applications. |
There was a problem hiding this comment.
"third-party" refers to another party, but it's perfectly fine to use a bundle made by your own company. What about:
This is no longer recommended and bundles should only be used to share code and features between multiple applications.
bundles.rst
Outdated
| * :doc:`/bundles/installation` | ||
| * :doc:`/bundles/remove` | ||
| * :doc:`/bundles/override` | ||
| * :doc:`/bundles/inheritance` |
There was a problem hiding this comment.
Let's stop linking to this - it'll still be findable by Google.
bundles.rst
Outdated
| :glob: | ||
|
|
||
| bundles/* | ||
| * :doc:`/bundles/installation` |
There was a problem hiding this comment.
And let's remove this link - I'm going to open a PR to remove this entirely
|
Woohooo, just another article updated! |
I've removed the
Bundle Directory Structurecontents ... but I plan to move them to other bundle articles in a separate PR.