Merge pull request #59 from cuviper/ci-merge

Try to make the CI status more robust
rust-num · Feb 1, 2024 · 9b4acd7 · 9b4acd7
2 parents ac0b695 + 5a97379
commit 9b4acd7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -43,9 +43,13 @@ jobs:
           components: rustfmt
       - run: cargo fmt --all --check
 
-  done:
+  ci-status:
     name: Complete
     runs-on: ubuntu-latest
     needs: [test, no_std, fmt]
+    if: always()
     steps:
-      - run: exit 0
+      - if: ${{ success() }}
+        run: exit 0
+      - if: ${{ !success() }}
+        run: exit 1
diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml
@@ -28,9 +28,13 @@ jobs:
           components: rustfmt
       - run: cargo fmt --all --check
 
-  done:
+  ci-status:
     name: Complete
     runs-on: ubuntu-latest
     needs: [test, fmt]
+    if: always()
     steps:
-      - run: exit 0
+      - if: ${{ success() }}
+        run: exit 0
+      - if: ${{ !success() }}
+        run: exit 1