Skip to content

Pin CodeQL SARIF action comment to v4.37.6 - #6313

Merged
samuv merged 1 commit into
mainfrom
fix/codeql-sarif-ref-comment
Aug 13, 2026
Merged

Pin CodeQL SARIF action comment to v4.37.6#6313
samuv merged 1 commit into
mainfrom
fix/codeql-sarif-ref-comment

Conversation

@samuv

@samuv samuv commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

The upload-sarif action is pinned to commit 5595ccaf91... but the trailing
version comment still said # v4, while that commit actually corresponds to
tag v4.37.6. zizmor's ref-version-mismatch audit flags this drift on every
PR/main run of the security-scan workflow.

  • Update the version comment on the github/codeql-action/upload-sarif pin
    in .github/workflows/security-scan.yml from # v4 to # v4.37.6 so it
    matches the tag the pinned commit actually points to.

Type of change

  • Bug fix

Test plan

  • Manual testing (describe below)

Verified the pinned commit 5595ccaf912efad79be6eef63a5619ff05969be3 is the
commit tagged v4.37.6 in github/codeql-action, matching zizmor's finding
and fix suggestion.

Does this introduce a user-facing change?

No.

Generated with Claude Code

The floating # v4 tag moved off the pinned commit, so zizmor
fails every PR with a ref-version-mismatch.
@samuv
samuv requested a review from JAORMX as a code owner August 13, 2026 16:06
@github-actions github-actions Bot added the size/XS Extra small PR: < 100 lines changed label Aug 13, 2026
@samuv samuv self-assigned this Aug 13, 2026
@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.87%. Comparing base (7854115) to head (3fcfde4).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6313      +/-   ##
==========================================
+ Coverage   72.82%   72.87%   +0.04%     
==========================================
  Files         742      742              
  Lines       77762    77804      +42     
==========================================
+ Hits        56632    56696      +64     
+ Misses      17155    17114      -41     
- Partials     3975     3994      +19     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@samuv
samuv merged commit daef21a into main Aug 13, 2026
30 checks passed
@samuv
samuv deleted the fix/codeql-sarif-ref-comment branch August 13, 2026 16:14
@github-actions github-actions Bot mentioned this pull request Aug 14, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants