Fix only-pnpm to support pnpm 11, and dedupe script#27332
Fix only-pnpm to support pnpm 11, and dedupe script#27332CraigMacomber wants to merge 6 commits into
Conversation
|
Hi! Thank you for opening this PR. Want me to review it? Based on the diff (97 lines, 8 files), I've queued these reviewers:
How this works
|
There was a problem hiding this comment.
Pull request overview
This PR updates pnpm enforcement logic and removes a duplicated compatibility-workspace copy of the pnpm-only install guard.
Changes:
- Updates the root
only-pnpm.cjscheck to usenpm_execpath. - Removes the compat-workspaces-local copy of
only-pnpm.cjs. - Points the compat workspace
preinstallscript at the root script.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
scripts/only-pnpm.cjs |
Updates pnpm detection logic. |
packages/test/test-version-utils/compat-workspaces/scripts/only-pnpm.cjs |
Removes duplicated pnpm enforcement script. |
packages/test/test-version-utils/compat-workspaces/full/package.json |
Redirects compat workspace preinstall to the root pnpm enforcement script. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Description
Fix only-pnpm to support pnpm 11, and dedupe script
Reviewer Guidance
The review process is outlined on this wiki page.