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

Licensing topic: 'npm config set key value' syntax reported not to work for npm version 9+ #3874

Open
ddaribo opened this issue Aug 30, 2023 · 1 comment

Comments

@ddaribo
Copy link
Contributor

ddaribo commented Aug 30, 2023

A couple of customers have reported that the
npm config set @infragistics:registry https://packages.infragistics.com/npm/js-licensed/

command is no longer working for them (using npm version 9+).
The command works with a = sign before the link though:

npm config set @infragistics:registry=https://packages.infragistics.com/npm/js-licensed/

The npm config documentation claims to support the npm config set key value syntax in version 9+ for backwards compatibility (link), however, as the customers report, this is not working for them.
Having this in mind, the command snippet should probably be updated?

@damyanpetev
Copy link
Member

Right, since 15.1.x is still in active support that means Angular 15 which still supports Node.js 14 https://angular.io/guide/versions#actively-supported-versions, which in turn is the last version that bundled npm 6 https://nodejs.org/en/download/releases.
Unfortunately, npm 6 from what I see was the one using the older API our docs use, so we can't quite replace it just yet. However, we could and probably should add the alternative - something in the lines of "For npm versions 7+" use the updated set call. I'd add we should test that works reliably from 7+ and if not list 8/9+ only.
//2c

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

No branches or pull requests

2 participants