Skip to content
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

pnpm config set registry --location=global fails in version 7.20.0 #5841

Closed
morsh opened this issue Dec 26, 2022 · 5 comments · Fixed by #5842
Closed

pnpm config set registry --location=global fails in version 7.20.0 #5841

morsh opened this issue Dec 26, 2022 · 5 comments · Fixed by #5842
Assignees
Milestone

Comments

@morsh
Copy link

morsh commented Dec 26, 2022

pnpm version:

7.20.0

Code to reproduce the issue:

pnpm config set registry https://pkgs.dev.azure.com/dynamicscrm/OneCRM/_packaging/ci-common-ui/npm/registry/ --location=global

Expected behavior:

Works

Actual behavior:

Unknown option: 'location'

Additional information:

  • node -v prints:
  • Windows, macOS, or Linux?:
@zkochan
Copy link
Member

zkochan commented Dec 26, 2022

We can fix this. But as an alternative, you can use --global instead of --location=global.

@morsh
Copy link
Author

morsh commented Dec 26, 2022

This was mainly a concern for us since it broke our build pipeline, --global fixes it, but I can't find any reference to any of the options in the documentation

@zkochan
Copy link
Member

zkochan commented Dec 26, 2022

In the past versions pnpm config was just passing through to npm. In v7.20 we added our own implementation. That is why it broke

@morsh
Copy link
Author

morsh commented Dec 26, 2022

So when I use --global it passes now, but it fails on:

 ERROR  Unable to find the global bin directory
Run "pnpm setup" to create it automatically, or set the global-bin-dir setting, or the PNPM_HOME env variable. The global bin directory should be in the PATH.

I'll try running pnpm setup - but again this is missing in the docs :)

@zkochan
Copy link
Member

zkochan commented Dec 26, 2022

OK, this one is also a bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants