Skip to content

Commit 3aa092f

Browse files
Fix CI failure due to missing coverage when tests are skipped
1 parent 20194cf commit 3aa092f

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

.github/workflows/test.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,16 +158,22 @@ jobs:
158158
name: coverage
159159
path: coverage/coverage-${{ steps.matrix-id.outputs.id }}.xml
160160

161-
combine:
161+
all-checks:
162162
if: ${{ always() }}
163163
runs-on: ubuntu-latest
164-
name: "All tests and coverage"
164+
name: "All tests"
165165
needs: [changes, style, test]
166166
steps:
167167
- name: Check build matrix status
168168
if: ${{ needs.changes.outputs.changes == 'true' && (needs.style.result != 'success' || needs.test.result != 'success') }}
169169
run: exit 1
170170

171+
upload-coverage:
172+
runs-on: ubuntu-latest
173+
name: "Upload coverage"
174+
needs: [changes, all-checks]
175+
if: ${{ needs.changes.outputs.changes == 'true' && needs.all-checks.result == 'success' }}
176+
steps:
171177
- uses: actions/checkout@v2
172178

173179
- name: Set up Python

0 commit comments

Comments
 (0)