Skip to content

Commit 5d5bda1

Browse files
authoredDec 18, 2024··
chore: Upgrade to Node 18 (#1586)
1 parent 53233c9 commit 5d5bda1

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+606
-402
lines changed
 

‎.github/workflows/ci.yml

+10-2
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ jobs:
110110
strategy:
111111
fail-fast: false
112112
matrix:
113-
node: [16, 22] # Min and max supported Node versions
113+
node: [18, 22] # Min and max supported Node versions
114114
platform: [linux-x64, linux-arm, macos-x64, macos-arm, windows-x64]
115115
include:
116116
- platform: linux-x64
@@ -129,6 +129,14 @@ jobs:
129129
- platform: windows-x64
130130
runner: windows-latest
131131
reuse-v8-context: true
132+
# Node 22.12.0 on Windows incorrectly resolves `localhost` to `::1`, rather than both `::1` and `127.0.0.1`.
133+
# We changed all of our internal tests to exclusively use `127.0.0.1`, but samples are still written to use
134+
# `localhost`, which really is the proper thing to do in samples. So until this gets fixed upstream, we force
135+
# the last known good version of Node on Windows.
136+
# See https://github.com/nodejs/node/issues/56137 (_resolved_ already, but not yet released).
137+
- platform: windows-x64
138+
node: 22
139+
node-release-override: 22.11.0
132140
runs-on: ${{ matrix.runner }}
133141
name: Run Integration Tests (${{ matrix.platform }}, Node ${{ matrix.node }}, Reuse V8 Context ${{ matrix.reuse-v8-context }})
134142
defaults:
@@ -155,7 +163,7 @@ jobs:
155163
- name: Install Node
156164
uses: actions/setup-node@v4
157165
with:
158-
node-version: ${{ matrix.node }}
166+
node-version: ${{ matrix.node-release-override || matrix.node }}
159167

160168
# On Windows, the 'runner.temp' variable uses backslashes as path separators, but
161169
# that may pose problems in later steps when we try to join that with subpaths;

‎.github/workflows/release.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ jobs:
210210
strategy:
211211
fail-fast: false
212212
matrix:
213-
node: [16, 22] # Min and max supported Node versions
213+
node: [18, 22] # Min and max supported Node versions
214214
platform: [linux-x64, linux-arm, macos-x64, macos-arm, windows-x64]
215215
sample: [hello-world, fetch-esm, hello-world-mtls]
216216
server: [cli, cloud]

0 commit comments

Comments
 (0)
Please sign in to comment.