-
Notifications
You must be signed in to change notification settings - Fork 788
feat: add autoflake, bandit, docformatter #992
base: main
Are you sure you want to change the base?
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
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.
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. |
2d42aac
to
50fa0e1
Compare
50fa0e1
to
84468e4
Compare
@timbedard thanks for working on this, any chance you can get it merged? :) i'm missing docformatter |
Here are some more Python tools.