fix(ci): fail requirements on failure/cancellation

maiieul · Jul 18, 2024 · 314cc9a · 314cc9a
1 parent 1d79edb
commit 314cc9a
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -731,15 +731,18 @@ jobs:
       - lint-package
       - build-docs
       - build-insights
-    if: |
-      always() &&
-      (needs.lint-package.result == 'success' || needs.lint-package.result == 'skipped') &&
-      (needs.test-e2e.result == 'success' || needs.test-e2e.result == 'skipped') &&
-      (needs.validate-rust.result == 'success' || needs.validate-rust.result == 'skipped') &&
-      (needs.build-docs.result == 'success' || needs.build-docs.result == 'skipped') &&
-      (needs.build-insights.result == 'success' || needs.build-insights.result == 'skipped')
+    if: always()
     steps:
-      - run: echo "All good :-)"
+      - name: check status
+        if: |
+          !(
+            (needs.lint-package.result == 'success' || needs.lint-package.result == 'skipped') &&
+            (needs.test-e2e.result == 'success' || needs.test-e2e.result == 'skipped') &&
+            (needs.validate-rust.result == 'success' || needs.validate-rust.result == 'skipped') &&
+            (needs.build-docs.result == 'success' || needs.build-docs.result == 'skipped') &&
+            (needs.build-insights.result == 'success' || needs.build-insights.result == 'skipped')
+          )
+        run: exit 1
 
   ############ RELEASE ############
   release: