Skip to content

chore: remove dead bandit configuration - #3203

Merged
IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
asarakhatun17-lgtm:fix-bandit-config
Sep 8, 2026
Merged

IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
asarakhatun17-lgtm:fix-bandit-config

Conversation

@asarakhatun17-lgtm

Copy link
Copy Markdown
Contributor

Removes the dead [tool.bandit] configuration and its commented-out pre-commit hook as requested in the issue.

@qodo-code-review

qodo-code-review Bot commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

Code Review by Qodo

Grey Divider

Sorry, something went wrong

We weren't able to complete the code review on our side. Please try again manually by commenting /agentic_review on this PR.

Grey Divider

Qodo Logo

@github-actions github-actions Bot added the chore label Sep 8, 2026

@IsmaelMartinez IsmaelMartinez left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, and that was a quick turnaround. You also took the right half of the choice: nothing installs bandit, it is not even in uv.lock, so that config could never have run and deleting it is the honest fix rather than wiring up a scan nobody asked for.

I checked the whole tree rather than just your two files. Those three references were the only ones, the suite is unchanged at 4362, pyproject.toml still parses with [tool.pytest.ini_options] and the ruff per-file ignores intact, and pre-commit passes on both files.

@IsmaelMartinez
IsmaelMartinez merged commit 349d106 into The-PR-Agent:main Sep 8, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants