You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In my case, I was in the root of a pnpm workspace, which had some global packages installed there. I'm using shamefully-hoist=true in my root .npmrc file. I had an NPM package installed from the NPM registry (installed with pnpm i -D <pkg>), then forked that packaged to make a contribution. I linked the forked package by running pnpm link --global and then went back to my pnpm workspace and ran pnpm link --global <pkg>, in my case in the root of a pnpm workspace.
Expected behavior:
package.json is not altered, the dependency stays a devDependency, as it already was.
Actual behavior:
After running all steps described above, the package was turned into a regular dependency.
Additional information:
node -v prints: v16.17.0
Windows, macOS, or Linux?: macOS
The text was updated successfully, but these errors were encountered:
pnpm link -g <pkg> uses the default config, which will always write pkg to dependencies.
Is it necessary to support option --save-prod/--save-dev here? Or add logic to read where the existing pkg locates? cc @zkochan
pnpm version:
7.13.4
Code to reproduce the issue:
In my case, I was in the root of a pnpm workspace, which had some global packages installed there. I'm using
shamefully-hoist=true
in my root.npmrc
file. I had an NPM package installed from the NPM registry (installed withpnpm i -D <pkg>
), then forked that packaged to make a contribution. I linked the forked package by runningpnpm link --global
and then went back to my pnpm workspace and ranpnpm link --global <pkg>
, in my case in the root of a pnpm workspace.Expected behavior:
package.json
is not altered, the dependency stays a devDependency, as it already was.Actual behavior:
After running all steps described above, the package was turned into a regular dependency.
Additional information:
node -v
prints: v16.17.0The text was updated successfully, but these errors were encountered: