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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add bandit maker #2557

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add bandit maker #2557

wants to merge 2 commits into from

Conversation

pesader
Copy link

@pesader pesader commented Nov 14, 2022

I added a maker for bandit, a tool to audit and detect common security problems in python code. When I was writing a test for it, I realized one of the tests was failing (false negative) so I fixed it as well. Each change is in a separate commit, for your reviewing convenience.

Thanks for developing this awesome plugin, I 100% depend on it! 馃殌

Add support to lint with bandit, a  tool that detects common security
issues on python code. Include tests to ensure everything is working.
Fix failing test, since python detects error as "parenthesis never
closed" instead of "unexpected end of line". As such, the error is now
detected in the '(' character, not in the '\n' caracter so decrement the
column number by one.

Also move the fixture to a specific directory, so it is more organized
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.

None yet

1 participant