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

#640 - Exit zero status based on severity level #715

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

maciejstromich
Copy link
Contributor

This PR enables more fine grained control over exit zero status based on severity level.

@maciejstromich
Copy link
Contributor Author

@ericwb @ghugo @sigmavirus24 @lukehinds the pep8 check is failing with bunch of

tests/unit/formatters/test_html.py:10:1: H216: The unittest.mock module should be used rather than the third party mock package unless actually needed. If so, disable the H216 check in hacking config and ensure mock is declared in the project's requirements.
import mock

how do you want to approach this? should this be a separate PR or can I include it in the current one?

@maciejstromich
Copy link
Contributor Author

maciejstromich commented Jun 12, 2021

ah apparently @ericwb created already a PR for that #685

bandit/cli/main.py Outdated Show resolved Hide resolved
bandit/cli/main.py Outdated Show resolved Hide resolved
maciejstromich and others added 3 commits December 12, 2021 14:32
Co-authored-by: Mikhail Feoktistov <zealotous@gmail.com>
Co-authored-by: Mikhail Feoktistov <zealotous@gmail.com>
@maciejstromich
Copy link
Contributor Author

@zealotous thanks a review.

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.

3 participants