Drop Node.js 14 for development build (#13439)
* Drop Node.js 16 for development * Fix engine * Address review
This commit is contained in:
1 parent
bd5f91545f
commit
e3ef1e4566
4 files changed
+8
-9
No files matched your search
@@ -22,7 +22,7 @@ jobs:
|
||||
- "ubuntu-latest"
|
||||
node:
|
||||
# Run tests on minimal version we support
|
||||
- "14"
|
||||
- "16"
|
||||
NPM_CLIENT:
|
||||
- "yarn"
|
||||
# Disabled temporarily #13151
|
||||
|
||||
@@ -21,7 +21,6 @@ jobs:
|
||||
node:
|
||||
- "18"
|
||||
- "16"
|
||||
- "14"
|
||||
include:
|
||||
# only enable coverage on the fastest job
|
||||
- os: "ubuntu-latest"
|
||||
@@ -29,11 +28,11 @@ jobs:
|
||||
ENABLE_CODE_COVERAGE: true
|
||||
FULL_TEST: true
|
||||
CHECK_TEST_PARSERS: true
|
||||
exclude:
|
||||
- os: "macos-latest"
|
||||
node: "16"
|
||||
- os: "windows-latest"
|
||||
node: "16"
|
||||
# exclude:
|
||||
# - os: "macos-latest"
|
||||
# node: "16"
|
||||
# - os: "windows-latest"
|
||||
# node: "16"
|
||||
env:
|
||||
ENABLE_CODE_COVERAGE: ${{ matrix.ENABLE_CODE_COVERAGE }}
|
||||
FULL_TEST: ${{ matrix.FULL_TEST }}
|
||||
|
||||
+1
-1
@@ -13,7 +13,7 @@
|
||||
"browser": "./standalone.js",
|
||||
"unpkg": "./standalone.js",
|
||||
"engines": {
|
||||
"node": ">=14"
|
||||
"node": ">=16"
|
||||
},
|
||||
"files": [
|
||||
"index.js",
|
||||
|
||||
@@ -134,7 +134,7 @@ async function preparePackage() {
|
||||
const packageJson = await readJson(path.join(PROJECT_ROOT, "package.json"));
|
||||
packageJson.bin = "./bin-prettier.js";
|
||||
// https://github.com/prettier/prettier/pull/13118#discussion_r922708068
|
||||
// packageJson.engines.node = ">=14.20.0";
|
||||
packageJson.engines.node = ">=14";
|
||||
delete packageJson.dependencies;
|
||||
delete packageJson.devDependencies;
|
||||
delete packageJson.browserslist;
|
||||
|
||||
Reference in new issue
Block a user