Merge pull request #111 from Nemo157/fix-workflow

Actually detect success of workflow
Nullus157 · May 23, 2023 · 0edad16 · 0edad16
2 parents 9147711 + fc6a9ca
commit 0edad16
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml
@@ -4,8 +4,14 @@ jobs:
   success:
     runs-on: ubuntu-latest
     needs: [pull_request]
+    if: ${{ always() }}
     steps:
-    - run: true
+    - if: ${{ contains(needs.*.result, 'failure')  }}
+      run: exit 1
+    - if: ${{ contains(needs.*.result, 'cancelled')  }}
+      run: exit 1
+    - if: ${{ contains(needs.*.result, 'skipped')  }}
+      run: exit 1
 
 on:
   pull_request:

diff --git a/.github/workflows/staging.yml b/.github/workflows/staging.yml
@@ -6,8 +6,14 @@ jobs:
   success:
     runs-on: ubuntu-latest
     needs: [pull_request, staging]
+    if: ${{ always() }}
     steps:
-    - run: true
+    - if: ${{ contains(needs.*.result, 'failure')  }}
+      run: exit 1
+    - if: ${{ contains(needs.*.result, 'cancelled')  }}
+      run: exit 1
+    - if: ${{ contains(needs.*.result, 'skipped')  }}
+      run: exit 1
 
 on:
   merge_group: