use conditional ifs instead of continue-of-error

OpenAstronomy · Jun 24, 2024 · a125690 · a125690
1 parent c5f222b
commit a125690
Showing 1 changed file with 17 additions and 14 deletions.
diff --git a/.github/workflows/tox.yml b/.github/workflows/tox.yml
@@ -249,23 +249,26 @@ jobs:
       - tox
     runs-on: ubuntu-latest
     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
+        continue-on-error: true
+      - if: steps.download.outputs.download-path != ''
+        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: steps.download.outputs.download-path != ''
+        run: python -Im pip install --upgrade coverage[toml]
+      - if: steps.download.outputs.download-path != ''
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+          lfs: true
+          submodules: ${{ inputs.submodules }}
+          ref: ${{ inputs.checkout_ref }}
+      - if: steps.download.outputs.download-path != ''
+        run: python -Im coverage combine
+      - if: steps.download.outputs.download-path != ''
+        run: python -Im coverage report -i -m --format=markdown >> $GITHUB_STEP_SUMMARY