Skip to content

Conversation

@gesta81
Copy link
Contributor

@gesta81 gesta81 commented Jan 26, 2026

move bandit config to pyproject.toml (otherwise it's a bit hidden in .pre-commit-config.yaml)

@tschm
Copy link
Member

tschm commented Jan 26, 2026

I am not sure about moving stuff out of .pre-commit-config.yaml. .pre-commit-config.yaml is part or can be part of the sync business by rhiza. We have currently no access or control over pyproject.toml. Would be interesting to get that...
@HarryCampion?

@gesta81
Copy link
Contributor Author

gesta81 commented Jan 26, 2026

I am not sure about moving stuff out of .pre-commit-config.yaml. .pre-commit-config.yaml is part or can be part of the sync business by rhiza. We have currently no access or control over pyproject.toml. Would be interesting to get that... @HarryCampion?

An alternative would be bandit.toml (similar to ruff.toml)

@tschm
Copy link
Member

tschm commented Jan 26, 2026

Yes, bandit.toml is a lot more in the spirit of rhiza. I know it's quite controversial to have all those little config files but they have a point!

@HarryCampion
Copy link
Collaborator

HarryCampion commented Jan 26, 2026

Not sure about another config file in the repo to manage, I think I prefer utilising the pyproject.toml (with the exception of ruff). We can add clear documentation on how to customise.

I think we can likely support both

https://bandit.readthedocs.io/en/latest/config.html

@tschm
Copy link
Member

tschm commented Jan 27, 2026

what happens if .pre-commit-config.yml points to pyproject.toml but pyproject.toml has no segment [tool.bandit]?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants