-
Notifications
You must be signed in to change notification settings - Fork 223
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
Fails when using Yarn v2 + PnP #66
Comments
Does |
Yes, you use const path = require('path');
const cliPath = require.resolve('@changesets/cli/package.json', { paths: [cwd] });
const changesetsCliPkgJson = require(cliPath); // or JSON.parse(fs.read*(cliPath) )
await exec('node', [path.resolve(cliPath, '../cli/bin.js'), cmd], { cwd }); You should always use something like this and not make assumptions about the |
Would you like to prepare a PR for this? |
This issue can be closed maybe? For me it works now with:
|
It looks like there's a line that explicitly references
node_modules
here: https://github.com/changesets/action/blob/master/src/run.ts#L186. When using Yarn PnP, this fails, since node_modules doesn't exist.The text was updated successfully, but these errors were encountered: