Skip to content

Commit

Permalink
Use Node 18 for GitHub workflows (#666)
Browse files Browse the repository at this point in the history
Node.js seems to have fixed a bunch of test runner issues v18.

Bumping the prerequisite to v18.
  • Loading branch information
minestarks authored Sep 8, 2023
1 parent 7f0b42a commit f36f033
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 11 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ jobs:
submodules: "true"
- uses: actions/setup-node@v3
with:
node-version: '16.17.1'
node-version: '18.17.1'
- name: npm install
run: npm install
- name: npm check
Expand Down Expand Up @@ -108,7 +108,7 @@ jobs:
python-version: '3.11'
- uses: actions/setup-node@v3
with:
node-version: '16.17.1'
node-version: '18.17.1'
- uses: Swatinem/rust-cache@v2
- name: Prereqs
run: python ./prereqs.py --install
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/publish-playground.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
python-version: '3.11'
- uses: actions/setup-node@v3
with:
node-version: '16.17.1'
node-version: '18.17.1'
- uses: Swatinem/rust-cache@v2
- name: Prereqs
run: python ./prereqs.py --install
Expand Down
4 changes: 0 additions & 4 deletions npm/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,3 @@ at <src/vs/base/common/cancellation.ts>. This code uses a simplified version of

Node.js tests can be run via `node --test` (see
<https://nodejs.org/dist/latest-v18.x/docs/api/test.html#test-runner-execution-model>).

The test module was also added to Node.js v16.17.0, and Electron 22 (which VS Code plans to move to
in first half of 2023) includes v16.17.1, so v16.17 should be our minimum Node.js
version supported (it shipped in Aug 2022).
2 changes: 1 addition & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
],
"license": "MIT",
"engines": {
"node": ">=16.17.0"
"node": ">=18.17.1"
},
"scripts": {
"eslint:check": "eslint -c ./.eslintrc.cjs --max-warnings 0 ./",
Expand Down
4 changes: 2 additions & 2 deletions prereqs.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@
python_ver = (3, 11) # Python support for Windows on ARM64 requires v3.11 or later
rust_ver = (1, 72) # Ensure Rust version 1.69 or later is installed
node_ver = (
16,
18,
17,
) # Node.js version 16.17 or later is required to support the Node.js 'test' module
)
wasmpack_ver = (0, 12, 1) # Latest tested wasm-pack version
rust_fmt_ver = (1, 6, 0) # Current version when Rust 1.72 shipped
clippy_ver = (0, 1, 69)
Expand Down

0 comments on commit f36f033

Please sign in to comment.