Skip to content
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

use new bundle strategy #186

Merged
merged 4 commits into from
Apr 27, 2024
Merged

Conversation

jrushlow
Copy link
Collaborator

@jrushlow jrushlow commented Apr 16, 2024

wip - see symfony/symfony-docs#19793

@jrushlow jrushlow added the Minor Minor Enhancement label Apr 16, 2024
@yceruto
Copy link

yceruto commented Apr 17, 2024

About the directory structure, there is one more possible change that matches the modern skeleton: the translations/ directory can be moved to the root level too

@jrushlow jrushlow added this to the 2.x - Pre-release milestone Apr 17, 2024
@jrushlow jrushlow mentioned this pull request Apr 17, 2024
18 tasks
@jrushlow
Copy link
Collaborator Author

jrushlow commented Apr 17, 2024

About the directory structure, there is one more possible change that matches the modern skeleton: the translations/ directory can be moved to the root level too

@yceruto - I was just thinking about the best way to handle that. And I believe it'll be done outside of this PR. reset-password && verify-email have the exact same translations - to keep from adding too much noise to this PR - I need to mv and add any missing translations that exist in one bundle but not the other.

@jrushlow
Copy link
Collaborator Author

@yceruto - I'm wondering if we could / should add array "shape" annotation(s) to https://github.com/symfony/symfony/blob/65ccca07588b108246a2aa3f5c8214e045c376be/src/Symfony/Component/HttpKernel/Bundle/AbstractBundle.php#L38 for better DX

PHPStan Squawk:
image

Fixed by: 089cec8

@jrushlow jrushlow changed the title wip - use new bundle strategy use new bundle strategy Apr 26, 2024
@jrushlow jrushlow added the Status: Needs Review Needs to be reviewed label Apr 26, 2024
Copy link
Member

@bocharsky-bw bocharsky-bw left a comment

Choose a reason for hiding this comment

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

Code looks good to me

@jrushlow jrushlow merged commit 5de06bc into SymfonyCasts:2.x Apr 27, 2024
6 checks passed
@jrushlow jrushlow deleted the minor/bundle-system branch April 27, 2024 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minor Minor Enhancement Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants