Add summary step

Addresses #70 #70 This lets us hook requirements to summary as opposed to dynamically named steps.
githedgehog · Nov 11, 2024 · f86c306 · f86c306
1 parent 76acb5a
commit f86c306
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 0 deletions.
diff --git a/.github/workflows/dev.yml b/.github/workflows/dev.yml
@@ -136,3 +136,18 @@ jobs:
         timeout-minutes: 60
         with:
           limit-access-to-actor: true
+
+  summary:
+    name: "summary"
+    if: ${{ always() }}
+    runs-on: "ubuntu-latest"
+    needs:
+      - build
+    steps:
+      - name: "Flag any build matrix failures"
+        if: ${{ needs.build.result != 'success' }}
+        run: |
+          >&2 echo "A critical step failed!"
+          exit 1
+
+
diff --git a/.github/workflows/sterile.yml b/.github/workflows/sterile.yml
@@ -196,3 +196,23 @@ jobs:
         timeout-minutes: 60
         with:
           limit-access-to-actor: true
+
+  summary:
+    name: "summary"
+    if: ${{ always() }}
+    runs-on: "ubuntu-latest"
+    needs:
+      - test
+      - push
+    steps:
+      - name: "Flag any test failures"
+        if: ${{ needs.test.result != 'success' }}
+        run: |
+          >&2 echo "One or more required tests failed"
+          exit 1
+      - name: "Flag any push failures"
+        if: ${{ needs.push.result != 'success' }}
+        run: |
+          >&2 echo "One or more required pushes failed"
+          exit 1
+