ci: pin setup-vp action to latest main commit#842
Conversation
Update voidzero-dev/setup-vp from the floating @v1 tag to the pinned latest commit on main (13e7afb), matching the repo convention of SHA-pinning actions with a version comment.
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis change pins the ChangesPin setup-vp Action to Commit SHA
Estimated code review effort: 1 (Trivial) | ~3 minutes Related PRs: None identified. Suggested labels: ci, dependencies Suggested reviewers: None identified. 🐰 A tag once loose, now locked in place, 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #842 +/- ##
=======================================
Coverage 94.24% 94.24%
=======================================
Files 10 10
Lines 747 747
Branches 235 235
=======================================
Hits 704 704
Misses 40 40
Partials 3 3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR hardens CI supply-chain security by replacing the floating voidzero-dev/setup-vp@v1 reference with a specific commit SHA, aligning action usage with a “pin by SHA” approach across workflows.
Changes:
- Pin
voidzero-dev/setup-vpto commit13e7afb99c66525824db54e107d667216e795d37instead of@v1. - Apply the pin consistently across CI (
nodejs.yml), packaging PR workflow (pkg.pr.new.yml), and publishing workflow (release.yml). - Add an inline comment annotating the pinned reference.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .github/workflows/release.yml | Pins setup-vp in the release workflow to a specific commit SHA. |
| .github/workflows/pkg.pr.new.yml | Pins setup-vp in the pkg-pr-new workflow to a specific commit SHA. |
| .github/workflows/nodejs.yml | Pins setup-vp in both CI jobs to a specific commit SHA. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| - name: Setup Vite+ | ||
| uses: voidzero-dev/setup-vp@v1 | ||
| uses: voidzero-dev/setup-vp@13e7afb99c66525824db54e107d667216e795d37 # main |
|
|
||
| - name: Setup Vite+ | ||
| uses: voidzero-dev/setup-vp@v1 | ||
| uses: voidzero-dev/setup-vp@13e7afb99c66525824db54e107d667216e795d37 # main |
|
|
||
| - name: Setup Vite+ | ||
| uses: voidzero-dev/setup-vp@v1 | ||
| uses: voidzero-dev/setup-vp@13e7afb99c66525824db54e107d667216e795d37 # main |
|
|
||
| - name: Setup Vite+ | ||
| uses: voidzero-dev/setup-vp@v1 | ||
| uses: voidzero-dev/setup-vp@13e7afb99c66525824db54e107d667216e795d37 # main |
Update
voidzero-dev/setup-vpfrom the floating@v1tag to the pinned latest commit onmain(13e7afb, feat: resolve Vite+ version from package.json / catalog / lockfile #102).This matches the repo convention of SHA-pinning actions with a version comment. Applied across
nodejs.yml,release.yml, andpkg.pr.new.yml(4 references).Summary by CodeRabbit