Skip to content

Pin actions - #16

Closed
raviqqe wants to merge 1 commit into
mainfrom
chore/pin-action
Closed

Pin actions#16
raviqqe wants to merge 1 commit into
mainfrom
chore/pin-action

Conversation

@raviqqe

@raviqqe raviqqe commented Jun 19, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings June 19, 2026 04:15
@raviqqe
raviqqe enabled auto-merge (squash) June 19, 2026 04:15

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

This PR improves the security and reproducibility of this repository’s CI/CD by pinning all GitHub Actions in existing workflows to immutable commit SHAs (instead of floating tags).

Changes:

  • Pin actions/checkout to a specific commit SHA across test, lint, and release workflows.
  • Pin caching and helper actions (swatinem/rust-cache, raviqqe/cargo-cache, rust-lang/crates-io-auth-action, streetsidesoftware/cspell-action, raviqqe/markdown-link-check) to specific commit SHAs.
  • Add inline comments indicating the intended upstream release tag for each pinned SHA.

Reviewed changes

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

File Description
.github/workflows/test.yaml Pins checkout + rust-cache actions to SHAs for deterministic builds/tests.
.github/workflows/release.yaml Pins checkout, cargo cache, and crates.io auth actions to SHAs for safer releases.
.github/workflows/lint.yaml Pins checkout, rust-cache, spell-check, and link-check actions to SHAs for deterministic linting.

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

@raviqqe raviqqe closed this Jun 19, 2026
auto-merge was automatically disabled June 19, 2026 04:26

Pull request was closed

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