-
Notifications
You must be signed in to change notification settings - Fork 7
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
Support newer versions of pnpm > 8 #65
Comments
This would be awesome. Spend more than an hour to get CI in a new project running. Solution was deleting lock file and run |
@m8xp0w3r yeah this was my solution too |
I also ran into this as well. I converted my package.json file to use the new workspaces and catalog features that were added in v9.5 only to find that the builds no longer run in nx cloud. |
Hi folks, the base images provided by Nx Cloud has corepack enabled, so you should be able to set pnpm version via pnpm v8 is just the fallback value if one isn't predefined with corepack. |
Thanks for taking the time to respond. This got me unblocked. In case it's any help here is an example package.json with pnpm 9 that runs on the agents. {
"scripts": {
"preinstall": "npx only-allow pnpm"
},
"packageManager": "[email protected]",
"dependencies": {
"@monodon/rust": "catalog:"
},
"devDependencies": {
"@nx/eslint": "catalog:",
"@nx/web": "catalog:",
"@nx/workspace": "catalog:",
"@vitest/ui": "catalog:",
"eslint": "catalog:",
"nx": "catalog:",
"qwik-nx": "catalog:",
"vite": "catalog:",
"vitest": "catalog:",
"typescript": "catalog:"
}
} |
pnpm version should be configurable like node version.
In a project where pnpm-lock.yaml is already set you should be able to select your pnpm version used in the nx cloud workflow
This should be configurable:
https://github.com/nrwl/nx-cloud-workflows/blob/main/workflow-steps/install-node/main.js#L67
The text was updated successfully, but these errors were encountered: