Repository navigation
fix: gitleaks shall ignore uds-docs - #1527
Conversation
Greptile SummaryThis PR prevents the local secret-scanning hook from scanning the generated
Confidence Score: 5/5The PR appears safe to merge because the exclusion is limited to the generated, gitignored root-level documentation workspace. The configured path matches the exact workspace created by the documentation tasks, while the anchored expression avoids excluding similarly named paths elsewhere. Important Files Changed
Reviews (1): Last reviewed commit: "fix: gitleaks shall ignore uds-docs" | Re-trigger Greptile |
|
The more clean fix would be to have the hk gitleaks take in files: https://github.com/defenseunicorns/uds-cli/blob/main/hk.pkl#L21 something like this? |
I'm leaning on Codex here, this is my first contact with Suggested alternative in e3e1072 . Let me know what you think. |
Description
After running
uds run dev-docs, there is a directoryuds-docsleft behind (that is in.gitignore) that will be scanned bygitleaksvia the pre-commit hook invoked byhk. Commits cannot take place sincegitleakswill find objectionable matter inuds-docs.This change causes
gitleaksto skip the generated (and.gitignore'd)uds-docspath....
Related Issue
Fixes #CLI-321
Type of change
Checklist before merging
This was tested by running
uds run dev-docsand then trying to commit this PR change itself.