Skip to content

[DependencyInjection] Fixed public service use case #13057

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
Feb 19, 2020

Conversation

HeahDude
Copy link
Contributor

@HeahDude HeahDude commented Feb 5, 2020

Supersedes #12977 which was merged in 5.0, this time using a different approach and targeting 4.4.

Fixes #12978 and complements #12985.

Copy link
Contributor Author

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

Thanks @OskarStark for the review

@HeahDude HeahDude force-pushed the fix-public-services branch from abcf5b4 to 92a1fb5 Compare February 18, 2020 20:39
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.

❤️

@OskarStark
Copy link
Contributor

Thank you Jules.

OskarStark added a commit that referenced this pull request Feb 19, 2020
…hDude)

This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

[DependencyInjection] Fixed public service use case

Supersedes #12977 which was merged in 5.0, this time using a different approach and targeting 4.4.

Fixes #12978 and complements #12985.

Commits
-------

3694980 [DependencyInjection] Fixed public service use case
@OskarStark OskarStark merged commit 3694980 into symfony:4.4 Feb 19, 2020
@HeahDude HeahDude deleted the fix-public-services branch February 19, 2020 11:28
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.

Service container documentation (^4.2 and ^5.0) is wrong/misleading
4 participants