Skip to content

Revert "tools: remove flake8-bandit" #134

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Revert "tools: remove flake8-bandit" #134

wants to merge 1 commit into from
+149 −29

Conversation

onerandomusername
Copy link
Member

Reverts #132

This was actually running in pre-commit and with linting.

Its also useful. What I did not catch is, its the S_ errors. Not B errors.

@onerandomusername onerandomusername added the skip changelog Skip the need for a changelog entry for a Pull Request label Jan 2, 2022
@onerandomusername onerandomusername force-pushed the revert-132-remove-bandit branch from 1542d21 to 0070000 Compare January 2, 2022 06:13
@coveralls
Copy link
Collaborator

Coverage Status

Coverage remained the same at 77.757% when pulling 0070000 on revert-132-remove-bandit into d47e097 on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shivansh-007 Shivansh-007 Awaiting requested review from Shivansh-007 Shivansh-007 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
skip changelog Skip the need for a changelog entry for a Pull Request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@onerandomusername @coveralls