Skip to content

renovate: Pin GitHub Action digests#32

Merged
bkeryan merged 1 commit into
mainfrom
users/bkeryan/renovate-pin-digests
May 23, 2025
Merged

renovate: Pin GitHub Action digests#32
bkeryan merged 1 commit into
mainfrom
users/bkeryan/renovate-pin-digests

Conversation

@bkeryan

@bkeryan bkeryan commented May 22, 2025

Copy link
Copy Markdown
Collaborator

What does this Pull Request accomplish?

Enable helpers:pinGitHubActionDigestsToSemver, which causes Renovate to do two things:

  • Replace actions/whatever@v3 with actions/whatever@<sha-hash> # v3
  • Replace actions/whatever@<sha-hash> # v3 with actions/whatever@<sha-hash> # v3.1.7

Why should this Pull Request be merged?

Pinning GitHub action digests improves build reproducibility and is a security best practice.

What testing has been done?

Tested in https://github.com/ni/nitypes-python/blob/main/.github/renovate.json

@github-actions

Copy link
Copy Markdown
Contributor

Test Results

 10 files  ±0   10 suites  ±0   8s ⏱️ -1s
 24 tests ±0   24 ✅ ±0  0 💤 ±0  0 ❌ ±0 
210 runs  ±0  210 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit df72576. ± Comparison against base commit cc5b1a6.

@bkeryan bkeryan enabled auto-merge (squash) May 22, 2025 23:42
@bkeryan bkeryan merged commit 17f9b2a into main May 23, 2025
14 checks passed
@bkeryan bkeryan deleted the users/bkeryan/renovate-pin-digests branch May 23, 2025 15:07
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