Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
remove the scope from the output of the container:debug command#11317
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
xabbuh commentedJul 5, 2014
| Q | A |
|---|---|
| Bug fix? | no |
| New feature? | yes |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | #11302 |
| License | MIT |
| Doc PR |
javiereguiluz commentedJul 5, 2014
I've checked the Symfony documentation and there are no listings for the output of the |
xabbuh commentedJul 5, 2014
@javiereguiluz Thanks for checking the docs. |
stof commentedJul 8, 2014
👍 |
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 would not removed the scope from non-text formats (YAML, XML, ...) as they mostly consumed by tools that might need this information.
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.
Fair point.
What do you think about the output of a single service? Would you like to keep the scope there too as@stof suggested?
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 think that makes sense.
xabbuh commentedJul 8, 2014
I updated the pull request to remove the scope information only in the list of services when the text format is used. |
fabpot commentedJul 23, 2014
👍 |
fabpot commentedJul 23, 2014
Thank you@xabbuh. |
…g command (xabbuh)This PR was merged into the 2.6-dev branch.Discussion----------remove the scope from the output of the container:debug command| Q | A| ------------- | ---| Bug fix? | no| New feature? | yes| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#11302| License | MIT| Doc PR |Commits-------f56b731 remove the scope from the output of the container:debug command