Skip to content

Commit b2a026e

Browse files
committed
ci: simplify test workflow to use Node 22.x only
- Remove Node version matrix (was testing 18.x and 20.x) - Update all jobs to use Node 22.x (current Active LTS) - Remove redundant matrix conditions from coverage steps Node 18 reached EOL in April 2025, and multi-version testing provides no value for a browser application where Node is only used for build tooling. https://claude.ai/code/session_01DXd2UwS1bSM3mGBHiS3Nrc
1 parent cbbb8b9 commit b2a026e

1 file changed

Lines changed: 5 additions & 13 deletions

File tree

.github/workflows/test.yml

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -16,18 +16,14 @@ jobs:
1616
runs-on: ubuntu-latest
1717
timeout-minutes: 10
1818

19-
strategy:
20-
matrix:
21-
node-version: [18.x, 20.x]
22-
2319
steps:
2420
- name: Checkout repository
2521
uses: actions/checkout@v4
2622

27-
- name: Setup Node.js ${{ matrix.node-version }}
23+
- name: Setup Node.js
2824
uses: actions/setup-node@v4
2925
with:
30-
node-version: ${{ matrix.node-version }}
26+
node-version: 22.x
3127
cache: "npm"
3228

3329
- name: Install dependencies
@@ -38,7 +34,6 @@ jobs:
3834

3935
- name: Run tests with coverage
4036
run: npm run test -- --run --coverage
41-
if: matrix.node-version == '20.x'
4237

4338
- name: Check coverage thresholds
4439
run: |
@@ -83,19 +78,17 @@ jobs:
8378
console.log('Coverage check skipped - no coverage data available');
8479
}
8580
"
86-
if: matrix.node-version == '20.x'
8781
8882
- name: Upload coverage report
8983
uses: actions/upload-artifact@v4
90-
if: matrix.node-version == '20.x'
9184
with:
9285
name: coverage-report
9386
path: coverage/
9487
retention-days: 30
9588

9689
- name: Comment coverage on PR
9790
uses: romeovs/lcov-reporter-action@v0.3.1
98-
if: github.event_name == 'pull_request' && matrix.node-version == '20.x'
91+
if: github.event_name == 'pull_request'
9992
with:
10093
github-token: ${{ secrets.GITHUB_TOKEN }}
10194
lcov-file: ./coverage/lcov.info
@@ -108,7 +101,6 @@ jobs:
108101
echo "\`\`\`" >> $GITHUB_STEP_SUMMARY
109102
cat coverage/coverage-summary.json 2>/dev/null | jq '.' || echo "Coverage summary not available"
110103
echo "\`\`\`" >> $GITHUB_STEP_SUMMARY
111-
if: matrix.node-version == '20.x'
112104
113105
lint:
114106
name: Code Quality
@@ -122,7 +114,7 @@ jobs:
122114
- name: Setup Node.js
123115
uses: actions/setup-node@v4
124116
with:
125-
node-version: 20.x
117+
node-version: 22.x
126118
cache: "npm"
127119

128120
- name: Install dependencies
@@ -149,7 +141,7 @@ jobs:
149141
- name: Setup Node.js
150142
uses: actions/setup-node@v4
151143
with:
152-
node-version: 20.x
144+
node-version: 22.x
153145
cache: "npm"
154146

155147
- name: Install dependencies

0 commit comments

Comments
 (0)