ci: replace node-build with build+deploy workflow (PR validation, corepack, frozen lockfile)#26
Conversation
- Runs build on every PR (currently the workflow only runs on push to main) - corepack honours the packageManager pnpm pin (current CI globally installs latest pnpm) - pnpm install --frozen-lockfile catches lockfile drift - Deploy step gated to main only, so PRs can never deploy - pnpm store caching via setup-node
Removes `cache: pnpm` from setup-node. The pnpm cache was the vehicle in the @tanstack/* supply-chain incident (https://tanstack.com/blog/incident-followup); for a small static- site repo the ~30s install cost is worth the simpler threat model.
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ 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 |
Summary
Replaces `node-build.yml` with a single `ci.yml` that splits build and deploy into separate jobs.
What changes
Notes for reviewers
Test plan