Skip to content
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

Added --quiet option to suppress no issues #89

Merged
merged 1 commit into from
Aug 24, 2022
Merged

Conversation

edvardm
Copy link
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

@edvardm edvardm force-pushed the master branch 2 times, most recently from 4433dc0 to 3ab84bc Compare April 27, 2021 14:04
@coveralls
Copy link

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

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
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
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