Skip to content

Commit f36f033

Browse files
authored
Use Node 18 for GitHub workflows (#666)
Node.js seems to have fixed a bunch of test runner issues v18. Bumping the prerequisite to v18.
1 parent 7f0b42a commit f36f033

File tree

6 files changed

+7
-11
lines changed

6 files changed

+7
-11
lines changed

.github/workflows/ci.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ jobs:
7676
submodules: "true"
7777
- uses: actions/setup-node@v3
7878
with:
79-
node-version: '16.17.1'
79+
node-version: '18.17.1'
8080
- name: npm install
8181
run: npm install
8282
- name: npm check
@@ -108,7 +108,7 @@ jobs:
108108
python-version: '3.11'
109109
- uses: actions/setup-node@v3
110110
with:
111-
node-version: '16.17.1'
111+
node-version: '18.17.1'
112112
- uses: Swatinem/rust-cache@v2
113113
- name: Prereqs
114114
run: python ./prereqs.py --install

.github/workflows/publish-playground.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ jobs:
4545
python-version: '3.11'
4646
- uses: actions/setup-node@v3
4747
with:
48-
node-version: '16.17.1'
48+
node-version: '18.17.1'
4949
- uses: Swatinem/rust-cache@v2
5050
- name: Prereqs
5151
run: python ./prereqs.py --install

npm/README.md

-4
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,3 @@ at <src/vs/base/common/cancellation.ts>. This code uses a simplified version of
6868

6969
Node.js tests can be run via `node --test` (see
7070
<https://nodejs.org/dist/latest-v18.x/docs/api/test.html#test-runner-execution-model>).
71-
72-
The test module was also added to Node.js v16.17.0, and Electron 22 (which VS Code plans to move to
73-
in first half of 2023) includes v16.17.1, so v16.17 should be our minimum Node.js
74-
version supported (it shipped in Aug 2022).

package-lock.json

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
],
88
"license": "MIT",
99
"engines": {
10-
"node": ">=16.17.0"
10+
"node": ">=18.17.1"
1111
},
1212
"scripts": {
1313
"eslint:check": "eslint -c ./.eslintrc.cjs --max-warnings 0 ./",

prereqs.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@
1515
python_ver = (3, 11) # Python support for Windows on ARM64 requires v3.11 or later
1616
rust_ver = (1, 72) # Ensure Rust version 1.69 or later is installed
1717
node_ver = (
18-
16,
18+
18,
1919
17,
20-
) # Node.js version 16.17 or later is required to support the Node.js 'test' module
20+
)
2121
wasmpack_ver = (0, 12, 1) # Latest tested wasm-pack version
2222
rust_fmt_ver = (1, 6, 0) # Current version when Rust 1.72 shipped
2323
clippy_ver = (0, 1, 69)

0 commit comments

Comments
 (0)