Skip to content

Conversation

@nathro
Copy link
Collaborator

@nathro nathro commented Sep 6, 2025

Update actions/checkout pin to v4.0.0

This PR downgrades the GitHub Actions actions/checkout used by our composite action to v4.0.0 (commit SHA pin) to address compatibility and policy alignment concerns.

Key changes

  • Downgrade actions/checkout from v4.2.2 to v4.0.0, pinned by commit (1e31de5…) for supply‑chain safety
  • Continue using a commit SHA rather than a tag to maintain immutability
  • Applies only to the composite action step that checks out the PR head; no production/runtime code is affected

Context

Risk and impact

  • Low risk: CI-only change; behavior remains functionally equivalent for checkout of PR heads
  • Easy rollback: revert to prior pinned SHA (11bd719…, v4.2.2) if needed

🤖 This description was generated automatically. Please react with 👍 if it's helpful or 👎 if it needs improvement.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Change looks good. One small hardening tweak to consider for the checkout step.


🤖 Automated review complete. Please react with 👍 or 👎 on the individual review comments to provide feedback on their usefulness.

action.yml Outdated
steps:
- name: Checkout PR Head
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
uses: actions/checkout@1e31de5234b9f8995739874a8ce0492dc87873e2 # v4.0.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider adding persist-credentials: false to the checkout step to avoid writing the token to the local git config. This follows least-privilege best practices and is recommended when the workflow doesn’t need to push back to the repo.

@nathro nathro merged commit 35b344c into main Sep 7, 2025
1 check passed
@nathro nathro deleted the nathro_lower_checkout_version branch September 7, 2025 19:26
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