-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Upgrade Node.js to active LTS version (18.x) #7348
Conversation
|
||
# Check for updates at https://github.com/npm/cli/releases | ||
# This version should be compatible with the Node.js version declared above. See https://nodejs.org/en/download/releases as well | ||
ARG NPM_VERSION=8.19.4 | ||
ARG NPM_VERSION=9.5.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, it probably makes sense to use that latest stable version 👍 It looks like you remove the package lock (by accident?), would you mind taking a look at that? |
e5868f7
to
4e17bee
Compare
It looks like this is breaking one of our tests, would you mind taking a look at that? 🙇 |
- to eq("is-number@jonschlinkert/is-number#semver:^4.0.0")
+ to eq("is-number@github:jonschlinkert/is-number#semver:^4.0.0") |
d933407
to
fe29a8e
Compare
@jurre @andrejleitner I updated the PR - all tests are passing now |
This is required to support the latest Yarn versions, which have dropped 16.x support. |
Ping @jurre? |
Thanks for the headsup, let me roll this out 👍 |
Changes
Upgrades the Node.js version that dependabot uses to the latest active LTS (Long Term Support) version which is 18.x. https://nodejs.org/en
Why
Maintenance LTS
version of Node.js instead of aActive LTS
version. https://nodejs.dev/en/about/releases/engine-strict
enabled and want to avoidRelated issues / threads
.npmrc
with"engine-strict=true"
rule throwsEBADENGINE
error #4072 (comment)