Skip to content

fix(ci): replace invalid Actions pin with the commit SHA it resolves to - #1935

Open
MGPOCKY wants to merge 1 commit into
bytedance:mainfrom
MGPOCKY:fix/ci-invalid-action-ref
Open

fix(ci): replace invalid Actions pin with the commit SHA it resolves to#1935
MGPOCKY wants to merge 1 commit into
bytedance:mainfrom
MGPOCKY:fix/ci-invalid-action-ref

Conversation

@MGPOCKY

@MGPOCKY MGPOCKY commented Jul 31, 2026

Copy link
Copy Markdown

Problem

These workflow uses: pins are invalid as written. Each item below shows the current value, why it is wrong, and the correct ref that must be used instead.

1. .github/workflows/scorecard.yml

Current (invalid):

uses: github/codeql-action/upload-sarif@59c2c1ca13dff455ab22ad8157d6ab220447da29 # v4

Why this is wrong:

  • 59c2c1ca13dff455ab22ad8157d6ab220447da29 is not a commit SHA. It is the Git object SHA of an annotated tag (v4).
  • GitHub Actions does not accept annotated tag object SHAs in uses: pins (Commits API returns No commit found for SHA).
  • The correct pin is the commit SHA that tag v4 points to: 45cbd0c69e560cd9e7cd7f8c32362050c9b7ded2.

Correct pin:

uses: github/codeql-action/upload-sarif@45cbd0c69e560cd9e7cd7f8c32362050c9b7ded2 # v4

Test plan

  • Each updated uses: ref resolves as a commit via the GitHub Commits API
  • Relevant CI jobs on this branch look healthy

@netlify

netlify Bot commented Jul 31, 2026

Copy link
Copy Markdown

Deploy Preview for agent-tars-docs canceled.

Name Link
🔨 Latest commit 983cbb4
🔍 Latest deploy log https://app.netlify.com/projects/agent-tars-docs/deploys/6a6c9cbb8d0eea0008e2a0f7

@netlify

netlify Bot commented Jul 31, 2026

Copy link
Copy Markdown

Deploy Preview for tarko canceled.

Name Link
🔨 Latest commit 983cbb4
🔍 Latest deploy log https://app.netlify.com/projects/tarko/deploys/6a6c9cbbdc7c42000880fc6e

@CLAassistant

CLAassistant commented Jul 31, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@MGPOCKY
MGPOCKY marked this pull request as ready for review July 31, 2026 13:02
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