-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Bump NodeJS supported versions to ^16.16 || ^18.13 and add engines to package.json #8201
Changes from all commits
7ff43f4
53b178d
e2ab1fb
c890078
10b3223
2d05096
273d137
7935ed5
1d02ef0
8779cba
08622a8
7eabd34
cda3ff4
3cd33ac
8e91155
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
v16.13.0 | ||
v18.13.0 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,7 +31,7 @@ | |
}, | ||
"engine-strict": true, | ||
"engines": { | ||
"node": ">=14.13.1" | ||
"node": "^16.16.0 || ^18.13.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. By setting the engines on the specific node packages, they shall refuse to install if the node version does not match these versions. This is great for blocking installation on older unsupported versions, however the configuration of "^16.16.0 || ^18.13.0" means "Only node 16 or 18" - you're currently also blocking the installation of these packages on the currently supported node 19 (and the lts node 20 when it comes out in April). For these per-package engine fields I strongly think we should allow current and future versions for the sake of forwards compatibility - |
||
}, | ||
"os": [ | ||
"darwin", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,7 @@ | |
"shopify" | ||
], | ||
"engines": { | ||
"vscode": "^1.64.0" | ||
"node": "^16.16.0 || ^18.13.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This seems incorrect |
||
}, | ||
"categories": [ | ||
"Other" | ||
|
@@ -43,7 +43,6 @@ | |
}, | ||
"devDependencies": { | ||
"@shopify/polaris-tokens": "^6.3.0", | ||
"@types/node": "14.x", | ||
"@types/vscode": "^1.64.0", | ||
"@vscode/test-electron": "^2.1.2", | ||
"globby": "^11.1.0", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,6 +6,9 @@ | |
"author": "Shopify <[email protected]>", | ||
"homepage": "https://polaris.shopify.com", | ||
"repository": "https://github.com/Shopify/polaris", | ||
"engines": { | ||
"node": "^16.16.0 || ^18.13.0" | ||
}, | ||
"bugs": { | ||
"url": "https://github.com/Shopify/polaris/issues" | ||
}, | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -54,7 +54,7 @@ function generateConfig({output, targets, stylesConfig}) { | |
/** @type {import('rollup').RollupOptions} */ | ||
export default [ | ||
generateConfig({ | ||
targets: 'extends @shopify/browserslist-config, node 12.20', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This scared me but nothing seems to be bad since changing the version... |
||
targets: 'extends @shopify/browserslist-config, node 16.16.0', | ||
stylesConfig: { | ||
mode: 'standalone', | ||
output: 'styles.css', | ||
|
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.
We set the engines field now so this is no longer needed.