-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[DependencyInjection] Doc for Allow to choose an index for service locators #11042
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
Conversation
1b98149
to
e607796
Compare
Done, thanks @OskarStark |
e607796
to
6ebdddb
Compare
accec35
to
10cfeb0
Compare
The code is merged so I removed the label. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a lot of comments which were all fixed.
I think this one is good to merge.
Thanks @OskarStark |
I think we are good to go here. Note for the mergers: this must go in |
10cfeb0
to
d63c298
Compare
…or service locators (Anthony MARTIN) This PR was submitted for the master branch but it was merged into the 4.3 branch instead (closes #11042). Discussion ---------- [DependencyInjection] Doc for Allow to choose an index for service locators Here is the doc for the new feature implemented in : [symfony/symfony#30348](symfony/symfony#30348) that follow the feature [symfony/symfony#30257](symfony/symfony#30257) Commits ------- d63c298 [DependencyInjection] Doc for Allow to choose an index for service locator collection
It took us some time ... but this is finally merged! @XuruDragon thanks a lot for your contribution. |
No problem, thanks to you to finally merge it ;) |
Here is the doc for the new feature implemented in : symfony/symfony#30348 that follow the feature symfony/symfony#30257