Skip to content

Commit 45b962d

Browse files
committed
chore(workflow): revert & fix test matrix
1 parent 57b1d5a commit 45b962d

1 file changed

Lines changed: 11 additions & 13 deletions

File tree

.github/workflows/ci.yml

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -61,24 +61,22 @@ jobs:
6161
- uses: github/codeql-action/analyze@9e0d7b8d25671d64c341c19c0152d693099fb5ba # v4.35.5
6262

6363
test:
64-
name: NodeJS ${{ matrix.node-version }} on ${{ matrix.os }}
64+
name: NodeJS ${{ matrix.node.version }} on ${{ matrix.os }}
6565
runs-on: ${{ matrix.os }}
6666

6767
permissions:
6868
contents: read
6969

7070
strategy:
7171
matrix:
72-
node-version: [20.x, 22.x, 24.x]
7372
os: [ubuntu-latest, macos-26]
74-
# Shared default node options
75-
node-options: '--test-coverage-exclude="test/**/*.js"'
76-
77-
include:
78-
# Override default node options for node 20.x
79-
# The "--test-coverage-exclude" flag does not exist in node 20.x.
80-
- node-version: 20.x
81-
node-options: ''
73+
node:
74+
- version: 20.x
75+
options: ''
76+
- version: 22.x
77+
options: --test-coverage-exclude=test/**/*.js
78+
- version: 24.x
79+
options: --test-coverage-exclude=test/**/*.js
8280

8381
steps:
8482
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
@@ -88,7 +86,7 @@ jobs:
8886

8987
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
9088
with:
91-
node-version: ${{ matrix.node-version }}
89+
node.version: ${{ matrix.node.version }}
9290

9391
- name: Environment Information
9492
run: |
@@ -99,11 +97,11 @@ jobs:
9997
run: npm cit
10098
env:
10199
CI: true
102-
NODE_OPTIONS: ${{ matrix.node-options }}
100+
NODE_OPTIONS: ${{ matrix.node.options }}
103101

104102
- uses: codecov/codecov-action@b9fd7d16f6d7d1b5d2bec1a2887e65ceed900238 # v4.6.0
105103
if: success()
106104
with:
107-
name: ${{ runner.os }} node.js ${{ matrix.node-version }}
105+
name: ${{ runner.os }} node.js ${{ matrix.node.version }}
108106
token: ${{ secrets.CORDOVA_CODECOV_TOKEN }}
109107
fail_ci_if_error: false

0 commit comments

Comments
 (0)