Skip to content

chore(ci): pin actions in PR-triggered workflows#3194

Merged
kevinjqliu merged 1 commit intomainfrom
chore/zizmor-pr-triggered-workflows
Mar 28, 2026
Merged

chore(ci): pin actions in PR-triggered workflows#3194
kevinjqliu merged 1 commit intomainfrom
chore/zizmor-pr-triggered-workflows

Conversation

@kevinjqliu
Copy link
Copy Markdown
Contributor

@kevinjqliu kevinjqliu commented Mar 27, 2026

What changed

  • Pinned external GitHub Actions to full commit SHAs in PR-triggered workflows:
    • .github/workflows/check-md-link.yml
    • .github/workflows/codeql.yml
    • .github/workflows/license_check.yml
    • .github/workflows/python-ci-docs.yml
    • .github/workflows/python-ci.yml

Why

  • Fixes zizmor unpinned-uses findings by replacing mutable refs with immutable SHAs.
  • Reduces supply-chain risk from tag/branch drift while preserving workflow behavior.

Verification

  • uvx --from zizmor zizmor --offline .github/workflows/ --fix=all
  • Result: No findings to report.

@kevinjqliu kevinjqliu force-pushed the chore/zizmor-pr-triggered-workflows branch from 4a650b7 to 67c14fe Compare March 27, 2026 18:21
@Fokko
Copy link
Copy Markdown
Contributor

Fokko commented Mar 28, 2026

Thanks for pinning this @kevinjqliu

@kevinjqliu kevinjqliu merged commit d87fdc7 into main Mar 28, 2026
27 of 29 checks passed
@kevinjqliu kevinjqliu deleted the chore/zizmor-pr-triggered-workflows branch March 28, 2026 17:03
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