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
--exit-zero option #510
--exit-zero option #510
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice add -- I like this feature! Well done
0d1d284
to
6a23eb9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI feature required
This feature is necessary for a CI environment where only the report is required for SonarQube or any other tool. The --exit-zero parameter will prevent a pipeline from brokering and generating the report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good, however you'll also have to update https://github.com/PyCQA/bandit/blob/master/doc/source/man/bandit.rst with the new argument.
@ericwb I've added the missing docs to man page source. |
@ericwb the requested changes @maciejstromich has finished them. Is this feature available in the next version as soon as possible? : D it would be great to be able to implement it as soon as possible. |
@frankyjquintero for the time being you can just add
to your requirements.txt (that's the branch from which this PR is being made) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.