8000 [FrameworkBundle] Better output for user in ContainerDebugCommand by JhonnyL · Pull Request #18556 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[FrameworkBundle] Better output for user in ContainerDebugCommand #18556

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
Apr 20, 2016

Conversation

JhonnyL
Copy link
Contributor
@JhonnyL JhonnyL commented Apr 15, 2016
Q A
Branch? 3.0
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

Right now, the message To search for a specific service, re-run this command with a search term. (e.g. debug:container log) is displayed to the user whenever the command is being run. (Except when the search term is given).

But if a user runs e.g. debug:container --parameters. This message is out of scope since the user is looking for information about parameters, not services.

This PR will update the command to give better output to the user.

debug:container
Will keep current behaviour
debug:container --parameters
Will hint how to search for specific parameter
debug:container --tags
Will hint how to search for specific tag

@javiereguiluz
Copy link
Member

👍 I like this improvement. Thanks @JhonnyL

@aitboudad
Copy link
Contributor

👍

1 similar comment
@stof
Copy link
Member
stof commented Apr 20, 2016

👍

@nicolas-grekas
Copy link
Member

Thank you @JhonnyL.

@nicolas-grekas nicolas-grekas merged commit 313d8b1 into symfony:3.0 Apr 20, 2016
nicolas-grekas added a commit that referenced this pull request Apr 20, 2016
…Command (JhonnyL)

This PR was merged into the 3.0 branch.

Discussion
----------

[FrameworkBundle] Better output for user in ContainerDebugCommand

| Q             | A
| ------------- | ---
| Branch?       | 3.0
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

Right now, the message `To search for a specific service, re-run this command with a search term. (e.g. debug:container log)` is displayed to the user whenever the command is being run. (Except when the search term is given).

But if a user runs e.g. `debug:container --parameters`. This message is out of scope since the user is looking for information about parameters, not services.

This PR will update the command to give better output to the user.

`debug:container`
Will keep current behaviour
`debug:container --parameters`
Will hint how to search for specific parameter
`debug:container --tags`
Will hint how to search for specific tag

Commits
-------

313d8b1 [FrameworkBundle] Better output for user in ContainerDebugCommand
@fabpot fabpot mentioned this pull request May 3, 2016
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.

6 participants
0