use conditional ifs instead of continue-of-error

OpenAstronomy · Jun 24, 2024 · 44d93ed · 44d93ed
1 parent c5f222b
commit 44d93ed
Showing 1 changed file with 17 additions and 9 deletions.
diff --git a/.github/workflows/tox.yml b/.github/workflows/tox.yml
@@ -251,21 +251,29 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
           lfs: true
           submodules: ${{ inputs.submodules }}
           ref: ${{ inputs.checkout_ref }}
       - uses: actions/download-artifact@v4
+        id: download
         with:
           pattern: .coverage.*
           merge-multiple: true
-      - uses: actions/setup-python@v5
+      - run: |
+          [ "$(ls -A .coverage*)" ] && exit 0 || exit 1
+        continue-on-error: true
+      - if: success()
+        uses: actions/setup-python@v5
         with:
           python-version: "3.12"
-        continue-on-error: true
-      - run: python -Im pip install --upgrade coverage[toml]
-        continue-on-error: true
-      - run: python -Im coverage combine
-        continue-on-error: true
-      - run: python -Im coverage report -i -m --format=markdown >> $GITHUB_STEP_SUMMARY
-        continue-on-error: true
+      - if: success()
+        name: generate coverage report
+        run: |
+          python -Im pip install --upgrade coverage[toml]
+          python -Im coverage combine
+          python -Im coverage report -i -m --format=markdown >> $GITHUB_STEP_SUMMARY
+      - if: success()
+        uses: actions/upload-artifact@v4
+        with:
+          name: .coverage
+          path: .coverage