Only upload coverage HTML report on failure (#203)

adamchainz · Dec 4, 2021 · 7d70af8 · 7d70af8
1 parent 20b2526
commit 7d70af8
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -62,17 +62,15 @@ jobs:
         with:
           name: coverage-data
 
-      - name: Combine data
-        run: python -m coverage combine
-
-      - name: Generate HTML report
-        run: python -m coverage html --skip-covered --skip-empty
+      - name: Combine coverage and fail if it's <100%
+        run: |
+          python -m coverage combine
+          python -m coverage html --skip-covered --skip-empty
+          python -m coverage report --fail-under=100
 
       - name: Upload HTML report
+        if: ${{ failure() }}
         uses: actions/upload-artifact@v2
         with:
           name: html-report
           path: htmlcov
-
-      - name: Terminal report
-        run: python -m coverage report --fail-under=100