Skip to content

[Routing] make RoutingControllerPass and AttributeServicesLoader final #61512

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

Merged
merged 1 commit into from
Aug 25, 2025

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Aug 25, 2025

Q A
Branch? 7.4
Bug fix? no
New feature? no
Deprecations? no
Issues see #61492 (comment)
License MIT

@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit 3ab58a4 into symfony:7.4 Aug 25, 2025
11 of 12 checks passed
@Spomky
Copy link
Contributor

Spomky commented Aug 25, 2025

While I agree with it, itit a BC break as making a class final is not allowed for minor releases.
May I suggest to add @final instead and change it in 8.0?

@nicolas-grekas
Copy link
Member

(the classes are new to 7.4 @Spomky )

@Spomky
Copy link
Contributor

Spomky commented Aug 25, 2025

(the classes are new to 7.4 @Spomky )

Yes, I just realized 🫢.

@xabbuh xabbuh deleted the pr-61492 branch August 25, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants