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

support all options in .bandit #17

Merged
merged 1 commit into from
Aug 29, 2020
Merged

support all options in .bandit #17

merged 1 commit into from
Aug 29, 2020

Conversation

shapiromatron
Copy link
Contributor

@shapiromatron shapiromatron commented Aug 1, 2020

  • Updated flake8-bandit to support all .bandit config options including targets and excludes
  • cache config parsing so it can be re-used as needed

Breaking change:

  • python >= 3.6 required

I'm using a number of more modern python features, could of course rewrite with older methods but wanted to write the simple way first unless there are objections; for most projects I've worked on and other frequently used tools, it seems like python 3.6 is a reasonable minimum version (black, pydantic, etc).

Implements additional features as discussed in #3

@tylerwince
Copy link
Owner

Thanks @shapiromatron! Any chance you could add a couple tests for common .bandit configs?

@shapiromatron
Copy link
Contributor Author

Thanks @shapiromatron! Any chance you could add a couple tests for common .bandit configs?

Sure, how do I test? I didn't really understand how to do it with this project.

@shapiromatron
Copy link
Contributor Author

@tylerwince bump :) don't want this one to get stuck...

@tylerwince tylerwince merged commit 00ba2e4 into tylerwince:master Aug 29, 2020
@crimsonknave
Copy link

It doesn't appear that this has made it into a published version yet, is that correct?

@djmattyg007
Copy link

@tylerwince Would you be able to publish a new release?

@shapiromatron shapiromatron deleted the bandit-config branch August 8, 2021 10:02
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.

4 participants