Add trustPolicyExclude for specific packages#1680
Add trustPolicyExclude for specific packages#1680meghprkh wants to merge 1 commit intounovue:devfrom
Conversation
New pnpm feature which complains for a few packages. Add them as trusted packages while the ecosystem upgrades to new versions of these
📝 WalkthroughWalkthroughA new Changes
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@pnpm-workspace.yaml`:
- Around line 58-62: Update the trustPolicyExclude list so entries match actual
workspace deps: remove the nonexistent entry "koa@2.16.3"; change "semver@6.3.1"
to a workspace-compatible version such as "semver@7.7.3" (or the exact semver
range from your lockfile/ packages/cli); and replace "tailwindcss@3.4.18" with
the 4.x entry used by the main packages (e.g., "tailwindcss@^4.1.17" or the
exact 4.x version from your lockfile); keep "chokidar@4.0.3" as-is if it matches
the lockfile. Ensure each excluded spec exactly matches the versions/ranges in
the lockfile so the trustPolicyExclude entries are effective.
New pnpm feature which complains for a few packages. Add them as trusted packages while the ecosystem upgrades to new versions of these
🔗 Linked issue
Fixes #1668
❓ Type of change
📚 Description
To be fair my pnpm install succeeded but I could not build the project due to tsdown/rollup issues, so this is somewhat of a blind commit
📸 Screenshots (if appropriate)
📝 Checklist
Summary by CodeRabbit