Skip to content

Commit b9fbaa1

Browse files
committed
ci(verify): updated to follow modern conventions
1 parent 9e193c2 commit b9fbaa1

File tree

4 files changed

+9924
-4395
lines changed

4 files changed

+9924
-4395
lines changed

.github/workflows/test.yml

Lines changed: 27 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,14 @@ name: Test
88
types:
99
- opened
1010
- synchronize
11+
12+
permissions:
13+
contents: read
14+
15+
env:
16+
FORCE_COLOR: 1
17+
NPM_CONFIG_COLOR: always
18+
1119
jobs:
1220
test_matrix:
1321
strategy:
@@ -27,18 +35,30 @@ jobs:
2735
node-version: "${{ matrix.node-version }}"
2836
cache: npm
2937
- run: npm clean-install
30-
- run: "npm run test:ci"
31-
test:
38+
- run: npm test
39+
40+
test_dev:
3241
runs-on: ubuntu-latest
33-
needs: test_matrix
3442
steps:
3543
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
3644
- uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2
3745
with:
38-
node-version: "lts/*"
46+
node-version-file: .nvmrc
3947
cache: npm
4048
- run: npm clean-install
4149
- run: npm audit signatures
42-
- name: Ensure dependencies are compatible with the version of node
43-
run: npx ls-engines
44-
- run: npm run lint
50+
- run: npm test
51+
52+
test:
53+
runs-on: ubuntu-latest
54+
needs:
55+
- test_dev
56+
- test_matrix
57+
if: ${{ !cancelled() }}
58+
steps:
59+
- name: All matrix versions passed
60+
if: ${{ !(contains(needs.*.result, 'failure')) }}
61+
run: exit 0
62+
- name: Some matrix version failed
63+
if: ${{ contains(needs.*.result, 'failure') }}
64+
run: exit 1

0 commit comments

Comments
 (0)