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

NPM CLI is deprecating -global #245

Closed
Brantone opened this issue Jun 7, 2022 · 1 comment
Closed

NPM CLI is deprecating -global #245

Brantone opened this issue Jun 7, 2022 · 1 comment
Labels
Feature Request Enhancement to existing functionality or new functionality

Comments

@Brantone
Copy link
Contributor

Brantone commented Jun 7, 2022

🗣️ Foreword

Debated on this being "Feature request" or "Bug"

NPM has been talking about deprecating -global - this cookbook needs to account for that.

😦 Problem Statement

Recent releases of NPM CLI have fulfilled the deprecation - causing helper to fail since it prints the deprecation warning to stderr https://github.com/sous-chefs/nodejs/blob/main/libraries/nodejs_helper.rb#L25

Luckily NPM has reverted it, but not before some damage done : https://github.com/npm/cli/releases/tag/v8.12.1

❔ Possible Solution

Update helper to use --location=global instead

⤴️ Describe alternatives you've considered

Pinning to specific NPM CLI version, but that's short-sighted

➕ Additional context

npm/cli@8898710

@Brantone Brantone added the Feature Request Enhancement to existing functionality or new functionality label Jun 7, 2022
@Brantone Brantone changed the title NPM is deprecating -bloal NPM CLI is deprecating -global Jun 7, 2022
@Brantone
Copy link
Contributor Author

Brantone commented Jun 8, 2022

Another option is to pass environment along so the npm list can better/properly mimic what npm install will eventually do.

@ramereth ramereth linked a pull request Aug 8, 2022 that will close this issue
3 tasks
@damacus damacus closed this as not planned Won't fix, can't repro, duplicate, stale Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Enhancement to existing functionality or new functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants