Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[FrameworkBundle] Fix debug:container --show-arguments missing cases#21290
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
7ee5a3d to57b17f4Comparenicolas-grekas commentedJan 14, 2017
ping@Cydonia7 |
chalasr commentedJan 14, 2017
To reproduce: |
Cydonia7 commentedJan 14, 2017
Thank you for fixing this. To be honest, I don't understand this part of the code that well and did not think of testing per-service. I'm 👍 on this, the tests look good and are passing. |
fabpot commentedJan 15, 2017
Thank you@chalasr. |
… missing cases (chalasr)This PR was merged into the 3.3-dev branch.Discussion----------[FrameworkBundle] Fix debug:container --show-arguments missing cases| Q | A| ------------- | ---| Branch? | 3.3| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#20861 (comment)| License | MIT| Doc PR | n/aFixes the new `--show-arguments` option for per-service describing(i.e. `debug:container foo_service --show-arguments`, doesn't work at all right now).Commits-------57b17f4 [FrameworkBundle] Fix debug:container --show-arguments missing cases
Uh oh!
There was an error while loading.Please reload this page.
Fixes the new
--show-argumentsoption for per-service describing(i.e.
debug:container foo_service --show-arguments, doesn't work at all right now).