Skip to content

Commit c87e4a6

Browse files
authored
fix(tests): Avoid Node v22.12.0 on Windows (#402)
1 parent 2a44e13 commit c87e4a6

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@ jobs:
2626
nestjs-exchange-rates,
2727
timer-examples
2828
]
29+
include:
30+
- os: windows-latest
31+
node: 22
32+
node-release-override: 22.11.0
2933

3034
steps:
3135
- uses: actions/checkout@v4
@@ -38,7 +42,11 @@ jobs:
3842

3943
- uses: actions/setup-node@v4
4044
with:
41-
node-version: ${{ matrix.node }}
45+
# Node 22.12.0 on Windows incorrectly resolves `localhost` to `::1`, rather than both `::1` and `127.0.0.1`.
46+
# THis causes errors when executing tests. So until this gets fixed upstream, we force the last known good
47+
# version of Node on Windows.
48+
# See https://github.com/nodejs/node/issues/56137 (_resolved_ already, but not yet released).
49+
node-version: ${{ matrix.node-release-override || matrix.node }}
4250
# Comment out cache line when testing with act:
4351
# (Test command is: act --platform ubuntu-latest=lucasalt/act_base:latest)
4452
cache: 'pnpm'

0 commit comments

Comments
 (0)