Skip to content

Update alias_private.rst #16598

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
Closed
wants to merge 1 commit into from
Closed

Update alias_private.rst #16598

wants to merge 1 commit into from

Conversation

issamkhadiri1989
Copy link
Contributor

Hello,

I think even when using the Service ID, the package and version options are mendatory because when using deprecated: XXX a message error is shown informing that the attribute "package"/"version" of the "deprecated" option is missing
@issamkhadiri1989
Update alias_private.rst
1c1d4a2 
Hello, I think even when using the Service ID, the `package` and `version` options are mendatory because when using `deprecated: XXX` a message error is shown informing that the attribute "package"/"version" of the "deprecated" option is missing
@issamkhadiri1989
Copy link
Contributor Author

Hello,

Can anyone take a look at this PR please ?
@javiereguiluz
Copy link
Member

Hi @issamkhadiri1989! I'm really sorry we didn't handle your PR on time 😞 Meanwhile, other PRs fixed this as part of their changes. See https://github.com/symfony/symfony-docs/blob/5.4/service_container/alias_private.rst#deprecating-services

So, let's close this one as fixed. Thanks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Needs Review

3 participants

@issamkhadiri1989 @javiereguiluz @carsonbot