Skip to content

ci: pin Scorecards checkout action by commit - #1966

Merged
lukaszlenart merged 1 commit into
apache:mainfrom
mcc0nnell:ci/pin-scorecards-checkout
Sep 19, 2026
Merged

lukaszlenart merged 1 commit into
apache:mainfrom
mcc0nnell:ci/pin-scorecards-checkout

Conversation

@mcc0nnell

Copy link
Copy Markdown
Contributor

The Scorecards workflow already pins the scorecard, artifact upload, and SARIF upload actions to immutable commits, but actions/checkout still follows the mutable v7 tag.

That job has security-events: write and id-token: write, so pin checkout to the exact commit currently resolved by v7 (3d3c42e5aac5ba805825da76410c181273ba90b1, v7.0.1) as well.

No workflow behavior changes.

Validation:

  • actionlint .github/workflows/scorecards-analysis.yaml
  • git diff --check

Signed-off-by: Robert McConnell <robert@mcc0nnell.org>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The change is reviewed, narrowly scoped, and has no unresolved issues.

Review effort: Lite
Findings: None

What changed in this PR

Pins the Scorecards workflow checkout action to the immutable v7.0.1 commit.

Changes:

  • Replaces actions/checkout@v7 with commit 3d3c42e5....
  • Updates the version comment to 7.0.1.
File Description
.github/​workflows/​scorecards-analysis.yaml Pins the checkout action by commit.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@lukaszlenart
lukaszlenart merged commit c407f7f into apache:main Sep 19, 2026
13 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.

3 participants