Skip to content

[Console] Add higher verbosity levels #6066

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
Seldaek opened this issue Nov 19, 2012 · 4 comments
Closed

[Console] Add higher verbosity levels #6066

Seldaek opened this issue Nov 19, 2012 · 4 comments
Labels
BrowserKit Console

Comments

@Seldaek
Copy link
Member

Seldaek commented Nov 19, 2012

I think there should be new VERBOSITY_VERY_VERBOSE and VERBOSITY_DEBUG levels added to https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Console/Output/OutputInterface.php#L27 and the application should accept by default levels via -v, -vv and -vvv. Not sure if the last bit is already support by the Console component or not.

For information the verbosity is handled in the Application
@dlsniper
Copy link
Contributor

I'll create a PR for this but to be clear:
-vv -> VERBOSITY_DEBUG
-vvv -> VERBOSITY_VERY_VERBOSE

Right?

@Seldaek
Copy link
Member Author

Seldaek commented Nov 21, 2012

I would have seen it the other way, because to me DEBUG is the lowest level info you can get. I agree VERY_VERBOSE is a crappy name, but I couldn't come up with anything better, if you do feel free to pick something else :)

@dlsniper dlsniper mentioned this issue Nov 21, 2012
Closed
@tvlooy tvlooy mentioned this issue Mar 30, 2013
Closed
@tvlooy
Copy link
Contributor

tvlooy commented Apr 4, 2013

please ping me if this gets merged, I'll update the docs

@dlsniper
Copy link
Contributor

dlsniper commented Apr 5, 2013

@tvlooy thanks, when @fabpot will decide on how to name things out I'll do the doc PR as well but until then it's been quiet on this topic until now. Thanks for your help.

Regards.

dlsniper added a commit to dlsniper/symfony that referenced this issue Apr 11, 2013
@dlsniper
Added more verbosity levels. See GH symfony#6066
fe88059
@dlsniper dlsniper mentioned this issue Apr 11, 2013
Closed
@Seldaek Seldaek mentioned this issue Apr 25, 2013
Merged
fabpot added a commit that referenced this issue Apr 25, 2013
@fabpot
merged branch Seldaek/verbosityflags (PR #7841)
7d0b30d 
This PR was merged into the master branch.

Discussion
----------

[Console] Add more verbosity levels

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #6066
| License       | MIT
| Doc PR        | symfony/symfony-docs#2554

This adds new verbosity levels.

Replaces #7626 /cc @dlsniper

Commits
-------

02c2038 Fix tests
5327ec4 Add notes in UPGRADE and CHANGELOG
77c9791 [Console] Add tests and support for more types of inputs
54c1377 [Console] Handle new verbosity levels
e639686 Revert invalid changes
8f4d376 Removed unused options
5bb4163 Fixed tests
b62d35f Fix handling of --verbose=... and BC break
16cdb61 Added more verbosity levels
@fabpot fabpot closed this as completed May 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
BrowserKit Console
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fabpot @Seldaek @tvlooy @dlsniper