Skip to content

Scan commits for secrets with ggshield - #211

Merged
jmjava merged 1 commit into
mainfrom
chore/gg-r01-ggshield-pre-commit
Nov 7, 2026
Merged

jmjava merged 1 commit into
mainfrom
chore/gg-r01-ggshield-pre-commit

Conversation

@jmjava

@jmjava jmjava commented Nov 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add the official ggshield pre-commit hook (id: ggshield, rev v1.56.1) so secret scan pre-commit runs from .pre-commit-config.yaml.
  • Keep the existing check-added-large-files and local ruff hooks.
  • Credentials stay outside the repo. No API key, token, or .env is committed.

Test plan

  • python3 -m pre_commit validate-config .pre-commit-config.yaml (MemAvailable above 8 GiB)
  • Confirm a later commit with pre-commit installed runs the ggshield hook and that a missing GitGuardian login fails closed without a key in the tree

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@jmjava
jmjava merged commit 3bc19d6 into main Nov 7, 2026
8 checks passed
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.

1 participant