search

ansys / pre-commit-hooks

Ansys-developed pre-commit hooks for automating style and formatting
https://pre-commit-hooks.docs.ansys.com/
MIT License
5 stars 1 forks source link

chore: update code base to fix bandit warnings #209

Closed SMoraisAnsys closed 1 month ago

SMoraisAnsys commented 1 month ago

As title says.

SMoraisAnsys commented 1 month ago

@klmcadams I saw this while working on pyaedt (I had some leftover after the tests overall the license checker haha) I should start to work on #199 this week

klmcadams commented 1 month ago

@SMoraisAnsys Could you make a test case to show that it suppresses bandit warnings?

SMoraisAnsys commented 1 month ago

@SMoraisAnsys Could you make a test case to show that it suppresses bandit warnings?

I'm not sure to understand what you mean by that. Like creating an empty repo with the modified file and a dedicated workflow to see the changes ? Bellow are screenshots from bandit ouput (the first is the result from main branch, the second is the result from this PR) image image

klmcadams commented 1 month ago

Now I'm confused. I will message you