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

[BUG] unsupported engine warning only displays after initial install #4987

Open
2 tasks done
lukekarrys opened this issue Jun 2, 2022 · 0 comments
Open
2 tasks done
Labels
Bug thing that needs fixing Release 8.x work is associated with a specific npm 8 release

Comments

@lukekarrys
Copy link
Member

lukekarrys commented Jun 2, 2022

Is there an existing issue for this?

  • I have searched the existing issues

This issue exists in the latest npm version

  • I am using the latest npm

Current Behavior

❯ npm i npm@latest

added 1 package, and audited 202 packages in 2s

11 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

❯ npm ls node-gyp
npm-test@1.0.0 /Users/lukekarrys/Desktop/garbage/npm-test
└─┬ npm@8.12.0
  ├─┬ @npmcli/run-script@3.0.2
  │ └── node-gyp@9.0.0 deduped
  └── node-gyp@9.0.0


❯ npm i npm
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'node-gyp@9.0.0',
npm WARN EBADENGINE   required: { node: '^12.22 || ^14.13 || >=16' },
npm WARN EBADENGINE   current: { node: 'v12.13.0', npm: '8.12.0' }
npm WARN EBADENGINE }

up to date, audited 202 packages in 985ms

11 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

❯ npm ls node-gyp
npm-test@1.0.0 /Users/lukekarrys/Desktop/garbage/npm-test
└─┬ npm@8.12.0
  ├─┬ @npmcli/run-script@3.0.2
  │ └── node-gyp@9.0.0 deduped
  └── node-gyp@9.0.0


❯ npm i npm
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'node-gyp@9.0.0',
npm WARN EBADENGINE   required: { node: '^12.22 || ^14.13 || >=16' },
npm WARN EBADENGINE   current: { node: 'v12.13.0', npm: '8.12.0' }
npm WARN EBADENGINE }

up to date, audited 202 packages in 934ms

11 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

Expected Behavior

I expect to see the unsupported engines warning on the initial install.

@lukekarrys lukekarrys added Release 8.x work is associated with a specific npm 8 release Bug thing that needs fixing Needs Triage needs review for next steps and removed Needs Triage needs review for next steps labels Jun 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug thing that needs fixing Release 8.x work is associated with a specific npm 8 release
Projects
None yet
Development

No branches or pull requests

1 participant