Skip to content

* bandit/cli/main.py: Set log level to ERROR if -q option is passed.#941

Open
bje- wants to merge 5 commits intoPyCQA:mainfrom
bje-:main
Open

* bandit/cli/main.py: Set log level to ERROR if -q option is passed.#941
bje- wants to merge 5 commits intoPyCQA:mainfrom
bje-:main

Conversation

@bje-
Copy link
Copy Markdown

@bje- bje- commented Aug 13, 2022

Fixes #907

@sigmavirus24
Copy link
Copy Markdown
Member

I think the conversation on that issue is misinterpreting the documentation. I thought -q was repeatable. If it's repeated a second time, I would expect us to up the logging level to error (e.g., -qq) but I'm not sure I agree with this change as it stands. Especially with the log message that you seem to be taking issue with

@bje-
Copy link
Copy Markdown
Author

bje- commented Aug 15, 2022

You're right. I think this is a better solution.

Copy link
Copy Markdown

@CTimmerman CTimmerman left a comment

Choose a reason for hiding this comment

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

I think -d conflicts with -q because line 313 says parser instead of group.

@bje-
Copy link
Copy Markdown
Author

bje- commented Oct 14, 2022

I think -d conflicts with -q because line 313 says parser instead of group.

No, -q and -v are in a group of mutually exclusive options. It's fine as it is.

@bje-
Copy link
Copy Markdown
Author

bje- commented Mar 12, 2023

How can I progress with getting this merged, please?

@bje-
Copy link
Copy Markdown
Author

bje- commented Apr 9, 2025

@sigmavirus24, @ericwb, @lukehinds Can you review, please?

@bje-
Copy link
Copy Markdown
Author

bje- commented Feb 25, 2026

Ping?

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.

Unable to find qualified name

3 participants