8000 [DependencyInjection] Doc for Allow to choose an index for service locators by XuruDragon · Pull Request #11042 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

[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

Merged
merged 1 commit into from
Sep 24, 2019

Conversation

XuruDragon
Copy link

Here is the doc for the new feature implemented in : symfony/symfony#30348 that follow the feature symfony/symfony#30257

@javiereguiluz javiereguiluz changed the title [DependencyInjection] Doc for Allow to choose an index for service lo… [DependencyInjection] Doc for Allow to choose an index for service locators Mar 4, 2019
@javiereguiluz javiereguiluz added the Waiting Code Merge Docs for features pending to be merged label Mar 4, 2019
@XuruDragon
Copy link
Author

Done, thanks @OskarStark

@XuruDragon XuruDragon force-pushed the di-tagged-locator branch 2 times, most recently from accec35 to 10cfeb0 Compare March 11, 2019 10:52
@OskarStark OskarStark removed the Waiting Code Merge Docs for features pending to be merged label Mar 30, 2019
@OskarStark
Copy link
Contributor

The code is merged so I removed the label.

Copy link
Contributor
@OskarStark OskarStark left a 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.

@XuruDragon
Copy link
Author

Thanks @OskarStark

@OskarStark
Copy link
Contributor

I think we are good to go here.

Note for the mergers: this must go in 4.3 branch ❗️

@javiereguiluz javiereguiluz changed the base branch from master to 4.3 September 24, 2019 08:51
javiereguiluz added a commit that referenced this pull request Sep 24, 2019
…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
@javiereguiluz javiereguiluz merged commit d63c298 into symfony:4.3 Sep 24, 2019
@javiereguiluz
Copy link
Member

It took us some time ... but this is finally merged! @XuruDragon thanks a lot for your contribution.

@XuruDragon
Copy link
Author

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 ;)

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
0