Skip to content

fix: pin actions/checkout to SHA in pr-review.yml - #74

Merged
don-petry merged 3 commits into
mainfrom
claude/issue-51-20260508-1409
May 11, 2026
Merged

fix: pin actions/checkout to SHA in pr-review.yml#74
don-petry merged 3 commits into
mainfrom
claude/issue-51-20260508-1409

Conversation

@don-petry

@don-petry don-petry commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Pins actions/checkout@v5 to its commit SHA 93cb6efe18208431cddfb8368fd83d5badbf9bfd in .github/workflows/pr-review.yml
  • Addresses the action-pinning compliance finding from the weekly audit

Closes #51

Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated CI workflow to pin the repository checkout action to a specific commit, improving consistency and stability of automated checks.

Review Change Stack

Review Change Stack

Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 8, 2026 14:10
@coderabbitai

coderabbitai Bot commented May 8, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 66bf8dd6-6eac-4376-bfc5-90010fb1239a

📥 Commits

Reviewing files that changed from the base of the PR and between 979e2fb and c8afd5a.

📒 Files selected for processing (1)
  • .github/workflows/pr-review.yml

📝 Walkthrough

Walkthrough

The actions/checkout action in the PR review workflow is pinned from @v5 to a specific commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd) with no other workflow changes.

Changes

GitHub Actions Pinning

Layer / File(s) Summary
Action Pin Update
.github/workflows/pr-review.yml
The Checkout agent repo step pins actions/checkout from @v5 to commit SHA 93cb6efe18208431cddfb8368fd83d5badbf9bfd.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

  • Compliance: unpinned-actions-pr-review.yml #51: Compliance: unpinned-actions-pr-review.yml — This PR directly addresses the compliance finding by pinning the unpinned actions/checkout@v5 action to a specific commit SHA as required by the action-pinning policy.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: pinning actions/checkout to a SHA in pr-review.yml, which is the primary objective of the PR.
Linked Issues check ✅ Passed The PR successfully addresses issue #51 by pinning actions/checkout@v5 to commit SHA 93cb6efe18208431cddfb8368fd83d5badbf9bfd, fulfilling the action-pinning compliance requirement.
Out of Scope Changes check ✅ Passed All changes in the PR are within scope, consisting only of pinning the actions/checkout action to a specific SHA as required by issue #51.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-51-20260508-1409

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

Pull request overview

Pins actions/checkout in the PR review automation workflow to a specific commit SHA to satisfy the repository’s action-pinning compliance requirement (issue #51).

Changes:

  • Replace actions/checkout@v5 with a full commit SHA pin (93cb6efe18208431cddfb8368fd83d5badbf9bfd) in pr-review.yml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@donpetry-bot donpetry-bot 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.

Automated review — APPROVED ✓

Risk: LOW
Reviewed commit: 7a4967a220b6b2abd4800f4e065d28823addbe97
Review mode: triage-approved (single reviewer)

Summary

Single-line change pinning actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd) in .github/workflows/pr-review.yml. This is a standard security best practice that prevents supply chain attacks via tag mutation.

Linked issue analysis

Closes #51 — a compliance finding (action-pinning / error severity) from the weekly audit flagging actions/checkout@v5 as unpinned. The fix directly addresses the finding by pinning to the exact SHA while retaining a # v5 comment for readability.

Findings

  • ✅ SHA 93cb6efe18208431cddfb8368fd83d5badbf9bfd verified as the correct commit for actions/checkout tag v5
  • ✅ Version comment (# v5) retained for maintainability
  • ✅ No functional changes to the workflow logic
  • ✅ No security concerns

CI status

Check Status
Claude Code ✅ SUCCESS
CodeQL (actions) ✅ SUCCESS
CodeQL (javascript-typescript) ✅ SUCCESS
CodeQL (python) ✅ SUCCESS
CodeRabbit ✅ SUCCESS

Reviewed automatically by the PR-review agent (single-reviewer). Reply if you need a human review.

@github-actions

Copy link
Copy Markdown
Contributor

Auto-rebase blocked — the base branch contains .github/workflows/ changes that require the workflows permission to merge into this branch, but the auto-rebase workflow's token does not have that permission.

Please rebase this branch manually:

git fetch origin
git rebase origin/main
git push --force-with-lease

@sonarqubecloud

Copy link
Copy Markdown

@don-petry
don-petry merged commit b49ac9a into main May 11, 2026
15 of 16 checks passed
@don-petry
don-petry deleted the claude/issue-51-20260508-1409 branch May 11, 2026 20:53
don-petry added a commit that referenced this pull request Jun 7, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
don-petry added a commit that referenced this pull request Jun 7, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
don-petry added a commit that referenced this pull request Jun 8, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
don-petry added a commit that referenced this pull request Jun 12, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
don-petry added a commit that referenced this pull request Jun 14, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
don-petry added a commit that referenced this pull request Jun 18, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
don-petry added a commit that referenced this pull request Jun 25, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
don-petry added a commit that referenced this pull request Jun 25, 2026
Pin actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd)
to comply with the action-pinning policy. Closes #51

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Co-authored-by: Don Petry Bot <donpetry+bot@gmail.com>
Co-authored-by: Gemini CLI <gemini-cli@example.com>
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.

Compliance: unpinned-actions-pr-review.yml

3 participants