Merge pull request #1063 from sablier-labs/ci/fix-slack-notify

ci: fix slack notify
sablier-labs · Oct 13, 2024 · c6e056a · c6e056a
2 parents c3fa793 + b137111
commit c6e056a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/ci-deep.yml b/.github/workflows/ci-deep.yml
@@ -76,11 +76,11 @@ jobs:
       name: "Fork tests"
 
   notify-on-failure:
-    if: always()
+    if: failure()
+    needs: ["lint", "build", "test-unit", "test-integration", "test-invariant", "test-fork"]
     runs-on: "ubuntu-latest"
     steps:
       - name: "Send Slack notification"
-        if: failure() # Runs only if a failure occurs in any previous job
         uses: "rtCamp/action-slack-notify@v2"
         env:
           SLACK_CHANNEL: "#ci-notifications"

diff --git a/.github/workflows/ci-fork.yml b/.github/workflows/ci-fork.yml
@@ -32,11 +32,11 @@ jobs:
       name: "Utils tests"
 
   notify-on-failure:
-    if: always()
+    if: failure()
+    needs: ["lint", "build", "test-fork", "test-utils"]
     runs-on: "ubuntu-latest"
     steps:
       - name: "Send Slack notification"
-        if: failure() # Runs only if a failure occurs in any previous job
         uses: "rtCamp/action-slack-notify@v2"
         env:
           SLACK_CHANNEL: "#ci-notifications"