Skip to content

Commit

Permalink
Only apply npm workaround on windows and node 14
Browse files Browse the repository at this point in the history
  • Loading branch information
SavageCore committed Feb 14, 2023
1 parent 4c90cf7 commit 1ccb53b
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion .github/workflows/npm-test.yml
Expand Up @@ -33,10 +33,14 @@ jobs:
node-version: ${{ matrix.node_version }}
architecture: ${{ matrix.architecture }}
cache: "npm"
- name: Install latest npm
- name: Install latest npm with workaround for windows and node 14
# Fix for windows bug when installing npm 9 with node 14
# https://github.com/actions/setup-node/issues/623#issuecomment-1313381044
if: matrix.node_version == 14 && matrix.os == 'windows-latest'
run: npm i -g npm@8 && npm i -g npm@latest
- name: Install latest npm
if: matrix.node_version != 14 || matrix.os != 'windows-latest'
run: npm i -g npm@latest
- name: Clean install dependencies
run: npm ci
- name: Run tests
Expand Down

0 comments on commit 1ccb53b

Please sign in to comment.