Skip to content

ci: use full-length commit sha for github owned actions#407

Merged
Fdawgs merged 1 commit into
mainfrom
ci/sha
May 21, 2026
Merged

ci: use full-length commit sha for github owned actions#407
Fdawgs merged 1 commit into
mainfrom
ci/sha

Conversation

@Fdawgs
Copy link
Copy Markdown
Owner

@Fdawgs Fdawgs commented May 21, 2026

Use full-length commit sha for github owned actions just to be on safe side, especially after https://github.blog/security/investigating-unauthorized-access-to-githubs-internal-repositories/

Copilot AI review requested due to automatic review settings May 21, 2026 07:06
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-owned GitHub Actions in the repo’s workflows to full-length commit SHAs (instead of version tags) to reduce supply-chain risk, while retaining the human-readable version in inline comments.

Changes:

  • Update actions/checkout references to a full 40-char commit SHA (with # v6.0.2 comment).
  • Update actions/setup-node references to a full 40-char commit SHA (with # v6.4.0 comment).
  • Update github/codeql-action (init/analyze) references to a full 40-char commit SHA (with # v4.35.5 comment).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/codeql.yml Pins actions/checkout and github/codeql-action/* to full commit SHAs.
.github/workflows/ci.yml Pins actions/checkout and actions/setup-node to full commit SHAs for CI runs.
.github/workflows/cd.yml Pins actions/checkout and actions/setup-node to full commit SHAs for release/publish jobs.

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

@Fdawgs Fdawgs merged commit 025c6b4 into main May 21, 2026
21 checks passed
@Fdawgs Fdawgs deleted the ci/sha branch May 21, 2026 08:10
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.

2 participants