Replace npm-install action with npm ci - #13247
Conversation
✅ Docs preview has no changesThe preview was not built because there were no changes. Build ID: 546afaacb1308d68ac163fd9 ✅ AI Style Review — No Changes DetectedNo MDX files were changed in this pull request. Review Log: View detailed log
|
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (9)
📝 WalkthroughWalkthroughNine GitHub Actions workflows are updated to install Node dependencies by running ChangesCI Workflow Dependency Installation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Warning Review ran into problems🔥 ProblemsStopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a 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: |
This PR switches away from the
bahmutov/npm-installaction to plainnpm ci. If we want to use a cache,actions/setup-nodeincludes it and we'd rather leverage that instead. This should reduce one more potential attack vector in case this action caches differently.Summary by CodeRabbit