Skip to content

Added --quiet option to suppress no issues#89

Merged
fsouza merged 1 commit intoPyCQA:masterfrom
edvardm:master
Aug 24, 2022
Merged

Added --quiet option to suppress no issues#89
fsouza merged 1 commit intoPyCQA:masterfrom
edvardm:master

Conversation

@edvardm
Copy link
Copy Markdown
Contributor

@edvardm edvardm commented Apr 27, 2021

Added --quiet option, so that when checking lots of python files which are ok you don't get all those success messages printed. Other option could be to emit that no issues only with --verbose, but modifying existing things could break stuff

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 27, 2021

Coverage Status

Coverage increased (+0.004%) to 99.267% when pulling 7acde62 on EdvardM:master into 3c3b2a2 on myint:master.

@BenQuigley
Copy link
Copy Markdown

Good PR; it would be good to merge this. In the meantime I'm stuck running autoflake --check > /dev/null to suppress the stdout output. (Actually I think the quiet mode should be the default, and 'no problem' should only be logged if running with verbosity >= 1.)

@edvardm
Copy link
Copy Markdown
Contributor Author

edvardm commented Feb 23, 2022

Good PR; it would be good to merge this. In the meantime I'm stuck running autoflake --check > /dev/null to suppress the stdout output. (Actually I think the quiet mode should be the default, and 'no problem' should only be logged if running with verbosity >= 1.)

Agreed, this was a quick fix.

However, this project seems to be abandoned :( Or author is just too busy -- which is quite understandable. If that's the case, it would be great to have this information in README.md

Copy link
Copy Markdown
Collaborator

@fsouza fsouza left a comment

Choose a reason for hiding this comment

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

Thanks for contributing!

@fsouza fsouza merged commit bc62215 into PyCQA:master Aug 24, 2022
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