Skip to content

Update 3.3-di-changes.rst #9813

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 2 commits into from
May 27, 2018
Merged

Update 3.3-di-changes.rst #9813

merged 2 commits into from
May 27, 2018

Conversation

DonCallisto
Copy link
Contributor

@DonCallisto DonCallisto commented May 24, 2018

From symfony 3.4 aliases are not public by default https://symfony.com/blog/new-in-symfony-3-4-services-are-private-by-default so docs are not in synch with this change.

Maybe would be better to highlight this also in symfony 3.3 docs as a notice?

From symfony 3.4 aliases are not public by default https://symfony.com/blog/new-in-symfony-3-4-services-are-private-by-default so docs are not in synch with this change.

Maybe would be better to highlight this also in symfony 3.3 docs as a notice?
Moreover this seems to me nearly a BC.
@javiereguiluz javiereguiluz added this to the 3.4 milestone May 24, 2018
@javiereguiluz
Copy link
Member

Thanks Samuele.

@javiereguiluz javiereguiluz merged commit ec76e34 into symfony:3.4 May 27, 2018
javiereguiluz added a commit that referenced this pull request May 27, 2018
This PR was merged into the 3.4 branch.

Discussion
----------

Update 3.3-di-changes.rst

From symfony 3.4 aliases are not public by default https://symfony.com/blog/new-in-symfony-3-4-services-are-private-by-default so docs are not in synch with this change.

Maybe would be better to highlight this also in symfony 3.3 docs as a notice?

Commits
-------

ec76e34 Minor tweak
870e005 Update 3.3-di-changes.rst
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.

3 participants