-
-
Notifications
You must be signed in to change notification settings - Fork 204
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
Drop support for Node 14 #569
Conversation
package.json
Outdated
@@ -13,7 +13,7 @@ | |||
"type": "module", | |||
"exports": "./index.js", | |||
"engines": { | |||
"node": "^14.18.0 || ^16.14.0 || >=18.0.0" | |||
"node": "^16.14.0 || >=18.0.0 || >=20.0.0" |
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.
I am not sure the exact version range you'd prefer. Please let me know, and I can update it.
|
||
const escapeArg = arg => { | ||
if (typeof arg !== 'string' || NO_ESCAPE_REGEXP.test(arg)) { | ||
return arg; | ||
} | ||
|
||
return `"${arg.replace(DOUBLE_QUOTES_REGEXP, '\\"')}"`; | ||
return `"${arg.replaceAll('"', '\\"')}"`; |
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.
Linting-related updates due to xo
upgrade.
@@ -44,7 +44,7 @@ | |||
"dependencies": { | |||
"cross-spawn": "^7.0.3", | |||
"get-stream": "^6.0.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.
Upgrading get-stream
is not as straightforward, so I'll do it in a separate PR. This probably should be done before making the major release.
@@ -53,16 +53,16 @@ | |||
"strip-final-newline": "^3.0.0" |
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.
Upgrading signal-exit
is not as straightforward because one of the tests relies on internals for that library, so I'll do it in a separate PR.
This drops support for Node 14 (breaking change) since it is not maintained anymore.
This also adds CI tests for Node 20.
This also upgrades some dependencies with major releases.