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 no longer supports Node.js <4, stop updating it to the latest there #2677

Closed
mgol opened this issue Apr 26, 2017 · 4 comments
Closed

npm no longer supports Node.js <4, stop updating it to the latest there #2677

mgol opened this issue Apr 26, 2017 · 4 comments

Comments

@mgol
Copy link
Contributor

mgol commented Apr 26, 2017

Expected behaviour

npm shouldn't be updated to a version not supported in a particular environment.

Actual behaviour

npm gets updated to the latest version on all Node versions, including 0.10 & 0.12 which it no longer supports. This causes lots of warning messages and in some time it will most likely stop working:
https://travis-ci.org/karma-runner/karma/jobs/225953592

Environment Details

  • Karma version (output of karma --version): master
  • Relevant part of your karma.config.js file: N/A

Steps to reproduce the behaviour

  1. Open https://travis-ci.org/karma-runner/karma/jobs/225953592
  2. See warnings.
@mgol
Copy link
Contributor Author

mgol commented Apr 26, 2017

When is it planned to drop support for Node.js <4 in Karma? That would solve this issue for now.

If that's not planned soon, we should perhaps:

  1. Update npm to v3 in Node 0.10/0.12/4.
  2. Not touch npm in newer Nodes. This means we'll get stronger coverage as we'll be testing on Node 3 & 4 and soon on v5 as well - which is what people use as most of them don't update the built-in npm version.

@dignifiedquire
Copy link
Member

There is already a task open for that #2490. The main thing here is getting all modules ready for this for v2.0, which should include other breaking changes that we want to do.

@dignifiedquire
Copy link
Member

@mgol
Copy link
Contributor Author

mgol commented May 17, 2017

I think we should update npm in Node 0.10/0.12/4 to npm@3 as that we know is officially supported and leave the other Nodes as they are. Then we'll only test on npm>=3 but where such npm is already available by default we'll use the included version. That's the version most people will use so I think that's preferable.

PR: #2715.

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

No branches or pull requests

2 participants