Skip to content

chore(ci): bump github/codeql-action to v4.37.5 and sync the pin table (supersedes #61) - #66

Merged
neosun100 merged 1 commit into
mainfrom
fix/sync-pins-for-61
Aug 6, 2026
Merged

chore(ci): bump github/codeql-action to v4.37.5 and sync the pin table (supersedes #61)#66
neosun100 merged 1 commit into
mainfrom
fix/sync-pins-for-61

Conversation

@neosun100

Copy link
Copy Markdown
Contributor

Takes Dependabot #61's SHA bump for the three github/codeql-action sub-actions, with the authoritative table and version comments re-derived by tooling rather than hand-edited:

_AUTHORITATIVE  + d1ba80a13dd9 = v4.37.5
                - f205ea1c3313   (was v4.37.4, no longer pinned)
comments        codeql.yml:55, codeql.yml:68, scorecard.yml:65
                v4.37.4 -> v4.37.5

Produced by uv run python scripts/sync_action_pins.py --write (or make sync-action-pins).

Why #61 could not just be merged

Dependabot bumps the SHA but cannot update INV-CI-3's _AUTHORITATIVE table, so #61 failed CI on all four Pythons. That was the guard working correctly — an unrecorded pin must not slip through. What was missing was a supported fix: the guard's advice pointed at SENTINEL_VERIFY_ACTION_PINS=1, which passes in exactly that situation (it only validates entries already in the table). #65 added the tool; this is its first real use.

Verification

Closes #61 (superseded).

Takes Dependabot PR #61's SHA bump for the three github/codeql-action
sub-actions, with the authoritative table and the version comments re-derived by
`scripts/sync_action_pins.py --write` (INV-CI-5) rather than hand-edited:

  _AUTHORITATIVE  + d1ba80a13dd9 = v4.37.5
                  - f205ea1c3313   (was v4.37.4, no longer pinned)
  comments        codeql.yml:55, codeql.yml:68, scorecard.yml:65
                  v4.37.4 -> v4.37.5

This is the first real use of the tool added in #65, and the reason it exists:
PR #61 could not be merged as-authored because Dependabot cannot update the
table, and the guard's only advice was a flag that does not regenerate it.

Verified: guard failed on PR #61's tree, one command fixed it, guard passes.
Suite 4083 passed / 9 skipped. Zero unrelated files touched.
@neosun100
neosun100 merged commit 4160ede into main Aug 6, 2026
13 checks passed
@neosun100
neosun100 deleted the fix/sync-pins-for-61 branch August 6, 2026 14:17
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