-
Notifications
You must be signed in to change notification settings - Fork 2.6k
fix(core): convert * to workspace:* for pnpm/yarn/bun in CNW #33893
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for nx-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
View your CI Pipeline Execution ↗ for commit 02003c8
☁️ Nx Cloud last updated this comment at |
Templates use `"*"` for workspace dependencies in individual package.json files. This works for npm but breaks pnpm, yarn, and bun which require the `workspace:` protocol for proper symlinking. For pnpm, yarn, and bun: automatically convert `"*"` dependencies to `"workspace:*"` in all workspace package.json files. npm is left unchanged since it handles `"*"` natively. Also adds support for 2-level nested projects (e.g., `libs/shared/models/package.json`). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
|
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Templates use
"*"for workspace dependencies in individual package.json files. This works for npm but breaks pnpm, yarn, and bun which require theworkspace:protocol for proper symlinking.For pnpm, yarn, and bun: automatically convert
"*"dependencies to"workspace:*"in all workspace package.json files. npm is left unchanged since it handles"*"natively.Also adds support for 2-level nested projects (e.g.,
libs/shared/models/package.json).