feat(dev): add pre-commit hooks for code quality and security#74
Open
emmanuelgjr wants to merge 1 commit intoGenAI-Security-Project:v0.2from
Open
feat(dev): add pre-commit hooks for code quality and security#74emmanuelgjr wants to merge 1 commit intoGenAI-Security-Project:v0.2from
emmanuelgjr wants to merge 1 commit intoGenAI-Security-Project:v0.2from
Conversation
Add .pre-commit-config.yaml with: - pre-commit-hooks: trailing whitespace, end-of-file, YAML check, large files (500KB limit), merge conflict markers, debug statements - ruff: linting with auto-fix and formatting - bandit: security scanning (excludes tests) Add ruff configuration to pyproject.toml with rules for pycodestyle, pyflakes, isort, flake8-bugbear, and pyupgrade. Add bandit config excluding test directory. Add pre-commit and bandit to dev dependencies. Closes GenAI-Security-Project#51 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.pre-commit-config.yamlwith hooks for trailing whitespace, EOF fixer, YAML validation, large file detection (500KB), merge conflict markers, and debug statement detection[tool.ruff]and[tool.bandit]configuration topyproject.tomlpre-commitandbandit[toml]to dev dependenciesRuff rules enabled
EWFIBUPSetup for contributors
pip install -e ".[dev]" pre-commit installTest plan
pre-commitandbanditinstall successfully viapip install -e ".[dev]"pre-commit run --all-filespasses on the codebasepre-commit installsets up git hooks correctlyCloses #51
Generated with Claude Code