build: upload reports before maybe failing a goal check

nedbat · nedbat · commit 14aa78570110 · 2026-04-16T20:00:48.000-04:00
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -227,15 +227,6 @@ jobs:
           set -xe
           python igor.py combine_html
 
-      - name: "Check targets"
-        run: |
-          # Maybe someday goals will be a real feature
-          python -m pip install wcmatch
-          # Very crude check that nothing catastrophic has happened.
-          python lab/goals.py --group 90 "coverage/*.py" "tests/*.py"
-          # Absolutely all of our test code should be covered.
-          python lab/goals.py --file 100 "tests/test_*.py"
-
       - name: "Upload HTML report"
         uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7.0.0
         with:
@@ -256,6 +247,15 @@ jobs:
         run: |
           echo "total=$(python -m coverage report --format=total)" >> $GITHUB_OUTPUT
 
+      - name: "Check targets"
+        run: |
+          # Maybe someday goals will be a real feature
+          python -m pip install wcmatch
+          # Very crude check that nothing catastrophic has happened.
+          python lab/goals.py --group 90 "coverage/*.py" "tests/*.py"
+          # Absolutely all of our test code should be covered.
+          python lab/goals.py --file 100 "tests/test_*.py"
+
   publish:
     name: "Publish coverage report"
     needs: combine
