Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Console] Define isVerbose(), etc. methods in OutputInterface #13086

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

Closed

Conversation

frne
Copy link

@frne frne commented Dec 23, 2014

Could / should the Symfony\Component\Console\Output\OutputInterface define the methods isQuiet(), isVerbose(), isVeryVerbose() and isDebug()?

Because the Interface already defines the verbosity constants and the getVerbosity() method, it will probalby be the right place to do so...

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

@frne frne changed the title [Console] Define if*() verbosity methods in OutputInterface [Console] Define isVerbose(), etc. methods in OutputInterface Dec 23, 2014
@stof
Copy link
Member

stof commented Dec 23, 2014

👍 This was indeed on our list of changes for 3.0

@frne
Copy link
Author

frne commented Dec 23, 2014

@stof Didn't know. Where do I find that list?

@javiereguiluz
Copy link
Member

@frne here you have one list about proposed non-BC changes for 3.0: #11742

*
* @return bool true if verbosity is set to VERBOSITY_QUIET, false otherwise
*
* @api
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't add that yet.

@frne
Copy link
Author

frne commented Dec 31, 2014

Will reopen that once there is a 3.0 branch...

@frne frne closed this Dec 31, 2014
@xabbuh
Copy link
Member

xabbuh commented Dec 31, 2014

@frne Why did you close? master is the development branch for 3.0.

@stof stof reopened this Dec 31, 2014
@frne
Copy link
Author

frne commented Jan 2, 2015

@xabbuh @stof Sorry, obviously I misunderstood something ;)

@frne
Copy link
Author

frne commented Jan 2, 2015

@Tobion For my comprehension: You want me to remove the @api annotations? </dumb-question>

@fabpot fabpot added the Console label Jan 2, 2015
@Tobion
Copy link
Contributor

Tobion commented Jan 2, 2015

@frne yes

@fabpot
Copy link
Member

fabpot commented Jan 16, 2015

Thank you @frne.

fabpot added a commit that referenced this pull request Jan 16, 2015
…terface (frne)

This PR was squashed before being merged into the 3.0-dev branch (closes #13086).

Discussion
----------

[Console] Define isVerbose(), etc. methods in OutputInterface

Could / should the ```Symfony\Component\Console\Output\OutputInterface``` define the methods ```isQuiet()```, ```isVerbose()```, ```isVeryVerbose()``` and ```isDebug()```?

Because the Interface already defines the verbosity constants and the ```getVerbosity()``` method, it will probalby be the right place to do so...

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

Commits
-------

1ac95b1 [Console] Define isVerbose(), etc. methods in OutputInterface
@fabpot fabpot closed this Jan 16, 2015
@fabpot fabpot mentioned this pull request Nov 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants