Skip to content

Commit 355a862

Browse files
authored
chore(ci): Use yarn for node tests instead of npm (#2612)
This change removes usage of `npm` to install dependencies and run tests in the `node_tests.yml` CI workflow. Instead we now use `yarn` like in the rest of the repo.
1 parent dd8926f commit 355a862

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

.github/workflows/test_node.yml

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,9 @@ jobs:
2020
node-version-file: package.json
2121

2222
# We need to skip the fallback download because downloading will fail on release branches because the new version isn't available yet.
23-
- run: SENTRYCLI_SKIP_DOWNLOAD=1 npm install
23+
- run: SENTRYCLI_SKIP_DOWNLOAD=1 yarn install --frozen-lockfile
2424

25-
- run: npm run check:types
25+
- run: yarn check:types
2626

2727
test_node:
2828
strategy:
@@ -42,10 +42,13 @@ jobs:
4242
node-version: ${{ matrix.node-version }}
4343

4444
# We need to skip the fallback download because downloading will fail on release branches because the new version isn't available yet.
45-
- run: SENTRYCLI_SKIP_DOWNLOAD=1 npm install
45+
- run: SENTRYCLI_SKIP_DOWNLOAD=1 yarn install --frozen-lockfile
46+
if: matrix.node-version != '10.x' && matrix.node-version != '12.x'
4647

4748
# older node versions need an older nft
48-
- run: SENTRYCLI_SKIP_DOWNLOAD=1 npm install @vercel/nft@0.22.1
49+
- run: |
50+
SENTRYCLI_SKIP_DOWNLOAD=1 yarn install --ignore-engines --frozen-lockfile
51+
SENTRYCLI_SKIP_DOWNLOAD=1 yarn add @vercel/nft@0.22.1 --ignore-engines
4952
if: matrix.node-version == '10.x' || matrix.node-version == '12.x'
5053
51-
- run: npm test
54+
- run: yarn test

0 commit comments

Comments
 (0)