CI: Fix last unpinned action versions#35137
Conversation
There was a problem hiding this comment.
Pull request overview
Pins the remaining unpinned GitHub Action references to immutable commit SHAs, aligning these CI workflows/composite actions with the repo’s mandatory action version pinning policy.
Changes:
- Updates the composite
.github/actions/setup-node-and-installaction to use pinned SHAs foractions/setup-nodeandactions/cacherestore/save. - Normalizes the
actions/setup-nodeversion comment formatting ingenerate-sandboxes.ymlto match the repo’s# vX.Y.Zconvention.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/generate-sandboxes.yml | Keeps setup-node pinned and standardizes the inline version comment format. |
| .github/actions/setup-node-and-install/action.yml | Replaces tag-based action references (@v4) with pinned commit SHAs for setup-node and cache restore/save. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR pins three reused GitHub actions to specific commit hashes in the composite action setup workflow, replacing floating version references with deterministic commits for setup-node v6.4.0 and cache actions v5.0.5. The generate-sandboxes workflow is updated to reference the pinned setup-node commit. ChangesAction Version Pinning
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Comment |
Package BenchmarksCommit: The following packages have significant changes to their size or dependencies:
|
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 203 | 203 | 0 |
| Self size | 947 KB | 908 KB | 🎉 -39 KB 🎉 |
| Dependency size | 88.99 MB | 88.99 MB | 🚨 +360 B 🚨 |
| Bundle Size Analyzer | Link | Link |
We've enforced mandatory version pinning for actions.
These are the last outliers
Manual testing
Not needed.
Summary by CodeRabbit