Skip to content

Add note about the additional verbosity flags #2554

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 3 commits into from
May 4, 2013

Conversation

Seldaek
Copy link
Member

@Seldaek Seldaek commented Apr 25, 2013

No description provided.

fabpot added a commit to symfony/symfony that referenced this pull request Apr 25, 2013
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
@wouterj
Copy link
Member

wouterj commented Apr 25, 2013

I think the section added in #2408 should also get updated.

@Seldaek
Copy link
Member Author

Seldaek commented Apr 25, 2013

@wouterj thanks for the pointer, I updated the PR.

OutputInterface::VERBOSITY_VERBOSE Increased verbosity of messages
OutputInterface::VERBOSITY_VERY_VERBOSE Informative non essential messages
OutputInterface::VERBOSITY_DEBUG Debug messages
======================================= ==================================
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should add a versionnadded message for new ones

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done (I hope correctly, don't know much about the docs stuff)

weaverryan added a commit that referenced this pull request May 4, 2013
Add note about the additional verbosity flags
@weaverryan weaverryan merged commit 0f745f4 into symfony:master May 4, 2013
@weaverryan
Copy link
Member

Perfect, thanks guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants