deps: remove NPM lockfile and harden PNPM security rules#366
deps: remove NPM lockfile and harden PNPM security rules#366ondrej-langr wants to merge 7 commits into
Conversation
|
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 (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughUpdates workspace package manager to pnpm@11.1.2, adds a workspace "Security essentials" block to pnpm-workspace.yaml, and bumps ChangesWorkspace & Dependency Configuration Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
06146b2 to
9e6fb0e
Compare
|
@ondrej-langr can you resolve the merge conflict and I'll merge this |
|
sure thing |
# Conflicts: # pnpm-lock.yaml
|
should be resolved now |
The supply chain attacks happen more and more. PNPM offers some options that mitigate the attacks a lot. This change upgrades PNPM to latest and applies their recommendations about security settings - blockExoticSubdeps, minimumReleaseAge, trustPolicy, trustPolicyExclude and allowBuilds
Summary by CodeRabbit
Release Notes