fix: remove npm self-upgrade step crashing Release workflow#28
fix: remove npm self-upgrade step crashing Release workflow#28mynameistito merged 1 commit intomainfrom
Conversation
The 'npm install -g npm@latest' step fails with MODULE_NOT_FOUND: promise-retry because the GitHub Actions runner's cached npm 10.9.7 has an incomplete install. npm 10.9.7 already supports OIDC provenance (added in npm 9.5), so the upgrade is unnecessary.
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 44 minutes and 44 seconds. ⌛ 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: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ 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 |
|
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
1 participant
Problem
The Release workflow fails on the "Upgrade npm for OIDC support" step:
The GitHub Actions runner's cached npm 10.9.7 has an incomplete node_modules install (missing
promise-retry), sonpm install -g npm@latestcrashes because npm can't load its own dependencies to run.Fix
Remove the step entirely. npm 10.9.7 already supports OIDC/provenance publishing (that feature was added in npm 9.5.0), so the upgrade was unnecessary and just added fragility.
Test plan
Summary by cubic
Removed the
npm install -g npm@lateststep from the Release workflow to stop crashes caused by the runner’s corrupt cachednpm(missingpromise-retry). The existingnpm10.9.7 already supports OIDC/provenance, so no upgrade is needed.Written for commit 6dae066. Summary will update on new commits.
Note
Remove npm self-upgrade step crashing the Release workflow
The
npm install -g npm@lateststep in release.yml was causing the Release workflow to crash. Removing it means the workflow now uses whichever npm version the runner orsetup-nodeprovides.Macroscope summarized 6dae066.