-
Notifications
You must be signed in to change notification settings - Fork 369
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
plugin node_workspace doesn't work with pnpm? #1587
Comments
I don't recognize that error message Manifest plugins are abstracted in this repository and you should be able to register new plugin types if you control the execution environment of release-please (i.e. we run a GitHub bot on GCP that uses the As for the design of this library, we do not shell out to manipulate code as a design decision because we didn't want to add additional requirements on the runtime (like requiring additional binaries). If you'd like to contribute a |
Related: #1098 |
I'll do it in my spare time, so this means I have to manually modify pnpm-lock.yaml file right? |
I'm not sure exactly how a pnpm workspace works, but we do have updaters already for generic yaml files and workspace plugins can add additional files to update. |
I've run into the same problem. My Github action failed after logging "Building list of all packages". I must admit I did not run a debugger, but by analysing the code I found the following:
It seems lerna is working on pnpm support as we speak, which will most likely resolve the issue. |
awesome, I'll just wait then |
It might be useful to keep this issue open. Updating the lerna dependencies would be the minimum change required to address this once the lerna changes have been completed. |
ok |
Hello guys, I just noticed that the commit has been merged and lerna v5.5.0 supports pnpm. |
Hello, I have the same issue. I would assume that this PR will be resolved once it is merged, but what do you think? |
Hey, I'm currently using changesets in my pnpm monorepo, but I would like to switch to release-please. |
@chingor13 Merging #1449 will add support for this^^ |
Still doesn't work with pnpm. |
I'm trying to update local deps, monorepo is pnpm workspace
i tried to use plugin node_workspace, and got error:
I see that there is no way to write a custom manifest plugin
I did write a script, which's just simply 'cd' into each package folder, regex match workspace deps in package.json, and run 'pnpm update --workspace @scope/package1 @scope/package2'. The reason to run 'pnpm update' is to update pnpm-lock.yaml file also, for pnpm --frozen-lock option.
I'm not sure if this script fits your plugin system, if you want this, I can contribute
The text was updated successfully, but these errors were encountered: