feat: add autoflake, bandit, docformatter#992
feat: add autoflake, bandit, docformatter#992qbedard wants to merge 6 commits intojose-elias-alvarez:mainfrom
Conversation
| name = "bandit", | ||
| meta = { | ||
| url = "https://github.com/PyCQA/bandit", | ||
| description = "Bandit is a tool designed to find common security issues in Python code.", |
There was a problem hiding this comment.
one thing to mention in the description:
Since bandit doesn't get the original filename here, and is run via stdin, there is no way it respect can any excludes configured via a .bandit file.
( I had to hack around this temporarily until bandit accepts something like --stdin-filename).
There was a problem hiding this comment.
This definitely sounds like a good thing to mention, and we could add the workaround to the wiki and direct users there, too.
There was a problem hiding this comment.
@timbedard Let me know if you want to do anything about this (I think putting this on the wiki and linking to it would be helpful). Other than that I think this is good to go.
There was a problem hiding this comment.
I'm totally down. I've just been busy. If either of you get to it before me, cool. Otherwise I'll try to tackle it when I have some spare time.
jose-elias-alvarez
left a comment
jose-elias-alvarez
left a comment
There was a problem hiding this comment.
Looks great! Once we mention the bandit ignore issue in the documentation somewhere I think this is good to go.
|
with more time now, thank you @timbedard for adding this :) |
|
Let me know when you think this is okay! |
|
Thank you for your continued work on this PR! Everything looks good on my end, but again, let me know when you feel it's ready. |
qbedard
force-pushed
the
more-python-tools
branch
from
2d42aac to
50fa0e1
Compare
qbedard
force-pushed
the
more-python-tools
branch
from
50fa0e1 to
84468e4
Compare
|
@timbedard thanks for working on this, any chance you can get it merged? :) i'm missing docformatter |
6 participants
Here are some more Python tools.