From d7177b474d88041dc3aba3378f92332fb6eb3f8a Mon Sep 17 00:00:00 2001 From: relrelb Date: Mon, 31 Jan 2022 19:26:50 +0200 Subject: [PATCH] ci: Temporarily avoid npm 8.4 Upgrading to npm 8.4 on Windows is currently broken: https://github.com/npm/cli/issues/4341 This was avoided by disabling tests on Windows Node 14 in #6167. Re-enable them by staying on npm 8.3 instead. --- .github/workflows/test_web.yml | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/.github/workflows/test_web.yml b/.github/workflows/test_web.yml index 7537cbc86..9bad89853 100644 --- a/.github/workflows/test_web.yml +++ b/.github/workflows/test_web.yml @@ -34,11 +34,6 @@ jobs: node_version: ["14", "16"] rust_version: [stable] # We most likely don't care about Rust versions here, we'll catch those issues in test_rust.yml. os: [ubuntu-latest, windows-latest] - # TODO: Temporarily disable Node 14 on windows pending this issue: - # https://github.com/actions/setup-node/issues/411 - exclude: - - os: windows-latest - node_version: "14" steps: - uses: actions/checkout@v2 @@ -52,7 +47,8 @@ jobs: # Node.js 14 comes with npm 6, but we need npm 7 or newer. - name: Upgrade npm - run: npm install -g npm + # TODO: Temporarily avoid 8.4 due to: https://github.com/npm/cli/issues/4341 + run: npm install -g npm@8.3 - name: Install Rust toolchain uses: actions-rs/toolchain@v1