Skip to content

fix(ci): SHA-pin actions/checkout + actions/setup-python (closes #19)#20

Open
KYBvWHxW wants to merge 1 commit intomainfrom
fix/19-sha-pin-actions
Open

fix(ci): SHA-pin actions/checkout + actions/setup-python (closes #19)#20
KYBvWHxW wants to merge 1 commit intomainfrom
fix/19-sha-pin-actions

Conversation

@KYBvWHxW
Copy link
Copy Markdown
Contributor

Summary

Pin actions/checkout@v4 (4×) and actions/setup-python@v5 (4×) to commit SHAs across .github/workflows/ci.yml. Zero behavioural change — same commits the floating tags currently point to.

Why

Tags are mutable. Pinning to SHA is the enterprise standard per Gridltd-DevOps/.github#193. The SHA Pin Enforce ruleset will block any future PR touching this file otherwise.

Test Plan

  • YAML still parses cleanly
  • Same SHAs the floating tags currently resolve to — zero behavioural delta
  • CI passes on this branch

Related Issue

Closes #19. Parent: Gridltd-DevOps/.github#193 (cross-org SHA-pin audit).

🤖 Generated with Claude Code

Pin `actions/checkout@v4` and `actions/setup-python@v5` to commit
SHAs across 8 references in ci.yml. Zero behavioural change — same
commits the floating tags currently resolve to.

Surfaced by the cross-org SHA-pin audit
(Gridltd-DevOps/.github#193). The enterprise SHA Pin Enforce ruleset
would otherwise block any future PR touching this file.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 29, 2026 08:39
@KYBvWHxW
Copy link
Copy Markdown
Contributor Author

@MastLuo please review the SHA-pin fix when convenient.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Pins GitHub Actions used in the CI workflow to immutable commit SHAs to comply with SHA-pin enforcement rules (closes #19), while preserving the same action revisions currently referenced by the floating tags.

Changes:

  • Replace actions/checkout@v4 with actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 in all jobs.
  • Replace actions/setup-python@v5 with actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 in all jobs.
  • Add inline comments indicating the pinned major versions (# v4, # v5).

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

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.

fix(ci): SHA-pin actions/checkout + actions/setup-python in ci.yml (8 refs)

2 participants