Skip to content

Pin GitHub Actions to SHAs [SRE-1802]#18

Draft
tylergohl wants to merge 1 commit into
mainfrom
sre-1802-pin-actions-to-shas
Draft

Pin GitHub Actions to SHAs [SRE-1802]#18
tylergohl wants to merge 1 commit into
mainfrom
sre-1802-pin-actions-to-shas

Conversation

@tylergohl

@tylergohl tylergohl commented May 21, 2026

Copy link
Copy Markdown

Summary

Pin GitHub Actions references in this repo to immutable commit SHAs (with the original tag/branch preserved in a trailing comment). Part of SRE-1802 — org-wide supply-chain hardening.

Scope

This PR pins:

  • Internal ctrliq/* reusable workflow and action references
  • External actions not handled by clo-ciq's parallel Node 20 effort

Deferred to clo-ciq (see #17): actions/checkout, actions/setup-go.
Those will be SHA-pinned by clo's PR; this PR intentionally leaves them alone to avoid merge conflicts.

Files touched

  • .github/workflows/test.yaml

Test plan

  • Workflow runs succeed on this branch (or on next trigger after merge)
  • Once this and any companion clo-ciq PR merge, this repo has zero unpinned external/internal refs and sha_pinning_required can be enabled on it

Copilot AI review requested due to automatic review settings May 21, 2026 01:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Supply chain hardening for GitHub Actions by pinning action references to immutable commit SHAs (SRE-1802).

Changes:

  • Pinned golangci/golangci-lint-action in the test workflow to a specific commit SHA (with the v8 tag retained as an inline comment).

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

Comment on lines 19 to +20
- name: Go linter
uses: golangci/golangci-lint-action@v8
uses: golangci/golangci-lint-action@4afd733a84b1f43292c63897423277bb7f4313a9 # v8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants