Skip to content

Commit d0be849

Browse files
committed
fix: CI coverage upload conflicts and matrix handling
- Only run coverage on Node 20 (not all matrix nodes) - Use unique artifact names per node version - Add retention-days to prevent artifact buildup - Run regular tests on Node 18 and 22
1 parent b5bf18f commit d0be849

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,14 +33,20 @@ jobs:
3333
run: npm run build
3434

3535
- name: Run tests with coverage
36+
if: matrix.node-version == 20
3637
run: npm run test:coverage
3738

39+
- name: Run tests
40+
if: matrix.node-version != 20
41+
run: npm test
42+
3843
- name: Upload coverage report
44+
if: matrix.node-version == 20 && always()
3945
uses: actions/upload-artifact@v4
40-
if: always()
4146
with:
42-
name: coverage-report
47+
name: coverage-report-node-${{ matrix.node-version }}
4348
path: coverage/
49+
retention-days: 7
4450

4551
- name: Verify dist output
4652
run: |

0 commit comments

Comments
 (0)