-
Notifications
You must be signed in to change notification settings - Fork 40
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
Less verbose output #204
Comments
Nice idea. Happy to consider a pull request for this if anyone is interested. I suggest we follow the same approach as Ruff, that is we add a |
I'm not sure, is
❯ touch foobar.py
❯ ruff foobar.py
❯ ruff --quiet foobar.py Prints nothing (no violations). ❯ echo "blahlbha" > foobar.py
❯ ruff foobar.py
foobar.py:1:1: B018 Found useless expression. Either assign it to a variable or remove it.
foobar.py:1:1: F821 Undefined name `blahlbha`
Found 2 errors.
❯ ruff --quiet foobar.py
foobar.py:1:1: B018 Found useless expression. Either assign it to a variable or remove it.
foobar.py:1:1: F821 Undefined name `blahlbha` The In any case, just to clarify what this ticket suggests:
|
I don't understand, that seems to be what ruff is doing but I might be misunderstanding. In any case, feel free to outline here how you propose things should work in detail (maybe with some examples)? |
I mean that So I hope I sketched it out better this time! If not... we'll just carry on 😄 |
I don't think I'll do this unless as part of a major release. In the meantime, open to pull requests that add a |
Hi, thanks for the great library. Taking it into use in a legacy repository that we've been refactoring mercilessly! The layered mode will come to good use.
What would you think of option for less verbose output? Even with no violations, the output is quite verbose:
I'd rather only get single line (or no line at all, if all is okay).
We run
import-linter
alongside other tools, via a bash script. It will output something like:Most tools seem to default to not print anything at all, if there are no violations (only Black is more verbose as can be seen above). Would be nice to have the same behavior for
import-linter
, too?The text was updated successfully, but these errors were encountered: