[DependencyInjection] reuse the reflector tracked by the container builder #59350

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.

Sign up for GitHub

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

Jump to bottom

Merged

nicolas-grekas merged 1 commit into symfony:7.3 from xabbuh:pr-59088

Jan 5, 2025

Member

xabbuh commented Jan 2, 2025

Q	A
Branch?	7.3
Bug fix?	no
New feature?	no
Deprecations?	no
Issues
License	MIT

reworking the implementation from #59088 a bit


          reuse the reflector tracked by the container builder

b8b5fb4

carsonbot added Status: Needs Review DependencyInjection labels

carsonbot added this to the 7.3 milestone

Member

nicolas-grekas commented Jan 5, 2025

Thank you @xabbuh.

nicolas-grekas merged commit 16650e0 into symfony:7.3

11 checks passed

xabbuh deleted the pr-59088 branch

January 5, 2025 21:17

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

Labels

DependencyInjection Status: Needs Review