Skip to content

Commit b28a839

Browse files
committed
fix: address comments
1 parent 080baae commit b28a839

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

.github/workflows/code_health.yaml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
- name: Run tests
5353
run: npm test
5454
- name: Upload test results
55-
if: matrix.os == 'ubuntu-latest'
55+
if: always() && matrix.os == 'ubuntu-latest'
5656
uses: actions/upload-artifact@v4
5757
with:
5858
name: test-results
@@ -77,6 +77,7 @@ jobs:
7777
run: npm test -- --testPathIgnorePatterns "tests/integration/tools/mongodb" --testPathIgnorePatterns "tests/integration/[^/]+\.ts"
7878
- name: Upload test results
7979
uses: actions/upload-artifact@v4
80+
if: always()
8081
with:
8182
name: atlas-test-results
8283
path: coverage/lcov.info
@@ -102,12 +103,9 @@ jobs:
102103
with:
103104
name: atlas-test-results
104105
path: coverage/atlas
105-
- name: Install lcov-result-merger
106-
run: |
107-
npm install -g lcov-result-merger
108106
- name: Merge coverage reports
109107
run: |
110-
lcov-result-merger "coverage/*/lcov.info" "coverage/lcov.info"
108+
npx -y lcov-result-merger "coverage/*/lcov.info" "coverage/lcov.info"
111109
- name: Coveralls GitHub Action
112110
uses: coverallsapp/github-action@v2.3.6
113111
with:

0 commit comments

Comments
 (0)