Skip to content

Use modern way of loading the bundle's config by extending AbstractBu… - #7802

Open
michaelKaefer wants to merge 1 commit into
EasyCorp:5.xfrom
michaelKaefer:refactor/use-modern-bundle-loading
Open

Use modern way of loading the bundle's config by extending AbstractBu…#7802
michaelKaefer wants to merge 1 commit into
EasyCorp:5.xfrom
michaelKaefer:refactor/use-modern-bundle-loading

Conversation

@michaelKaefer

Copy link
Copy Markdown
Contributor

…ndle instead of Bundle

See https://symfony.com/doc/current/bundles/configuration.html which states:

There are two different ways of creating friendly configuration for a bundle:

Using the main bundle class: this is recommended for new bundles and for bundles following the recommended directory structure;
Using the Bundle extension class: this was the traditional way of doing it, but nowadays it's only recommended for bundles following the legacy directory structure.

And see https://symfony.com/doc/current/bundles/best_practices.html which states:

This directory structure is used by default when your bundle class extends the recommended AbstractBundle. If your bundle extends the Bundle class, you have to override the getPath() method ...

@michaelKaefer
michaelKaefer force-pushed the refactor/use-modern-bundle-loading branch 3 times, most recently from ca1b48d to 064641b Compare August 25, 2026 15:17
@michaelKaefer
michaelKaefer marked this pull request as draft August 25, 2026 15:17
@michaelKaefer
michaelKaefer force-pushed the refactor/use-modern-bundle-loading branch from 064641b to 85216d9 Compare August 25, 2026 15:20
@michaelKaefer
michaelKaefer marked this pull request as ready for review August 25, 2026 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant