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

Yesqa compatibility with bandit #136

Closed
lsorber opened this issue Aug 29, 2022 · 1 comment
Closed

Yesqa compatibility with bandit #136

lsorber opened this issue Aug 29, 2022 · 1 comment

Comments

@lsorber
Copy link

lsorber commented Aug 29, 2022

Hi, an issue I've come across is that yesqa will remove false positive # noqa statements that are actually necessary for bandit.

Is there a way to configure yesqa to ignore bandit noqas, or to otherwise take into account that another linter will run besides flake8?

I'm aware of the flake8-bandit plugin, but it hasn't been very dependable in my experience (e.g. it broke when flake8 v5.0 was released).

@asottile
Copy link
Owner

bandit should not use noqa as that's flake8's marker

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

No branches or pull requests

2 participants