Repository navigation
fix(ci): scope Gitleaks to checked-out history - #106
Merged
chrisknvidia merged 2 commits intoAug 27, 2026
Merged
Conversation
Signed-off-by: Rio Yu <[email protected]>
rioyu123
requested review from
chrisknvidia,
mosheabr,
rmalani-nv and
rng1995
as code owners
August 27, 2026 03:16
Signed-off-by: Rio Yu <[email protected]>
Collaborator
|
Looks good to me! |
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
Closes #105.
HEAD, instead of every unrelated ref fetched into the repository.--allaudits for scheduled and manually dispatched runs; unknown future event types also default to the wider audit.--full-historyand--diff-filter=tuxdbbehavior, terminate revision parsing with--, and keep the pinned image digest unchanged.HEADis not a valid commit. This matters because v8.30.0 can log an invalid-revision error while still returning exit code 0.For pull requests, GitHub's checked-out merge ref includes base ancestry and all PR commits. Pushes to
mainscan the fullmainancestry. Scheduled and manual runs retain the pre-PR repository-wide audit across fetched branches and tags.Verification
make lintmake test— see full-suite environment result belowmake buildFocused and static verification:
uv run pytest -q tests/test_ci_workflows.py— 11 passedmake PYTHON=.venv/bin/python lint— passedmake PYTHON=.venv/bin/python build— passed/usr/bin/codexwas installed and port 18080 was already occupied. Neither failing test intersects this diff.Pinned-image behavioral verification:
HEADscope scanned 65 commits, exited 0, and found no leaks;--allscanned 179 commits and detected the existing fixtures on other refs, confirming that scheduled/manual audit coverage is retained.HEAD: HEAD scope still exits 1.HEADremains unambiguous because the revision list ends with--.Release Impact
CHANGELOG.md