build: fix workflow syntax, maybe

Consensys · Sep 13, 2024 · 506920b · 506920b
1 parent c8a1541
commit 506920b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -94,11 +94,10 @@ jobs:
       - test
     uses: ./.github/workflows/slack-notifications.yml
     with:
-      status: ${{ job.status }}
+      status: ${{ needs.test.result }}
       actor: ${{ github.actor }}
       repository: ${{ github.repository }}
       pr_title: ${{ github.event.pull_request.title }}
       branch: ${{ github.event.pull_request.head.ref }}
       run_id: ${{ github.run_id }}
-      failures: ${{ steps.tests.outputs.failures }}
-
+      failures: ${{ needs.test.outputs.failures }}
diff --git a/.github/workflows/push.yml b/.github/workflows/push.yml
@@ -81,16 +81,16 @@ jobs:
         go test -v -p 4 -timeout=50m -tags=release_checks -short -race ./test/...
         go test -v -run=NONE -fuzz=FuzzIntcomp -fuzztime=30s ./internal/backend/ioutils
 
-  
   slack-notifications:
     if: always()
     needs:
       - staticcheck
       - test
     uses: ./.github/workflows/slack-notifications.yml
     with:
-      status: ${{ job.status }}
+      status: ${{ needs.test.result }}
       actor: ${{ github.actor }}
       repository: ${{ github.repository }}
       branch: ${{ github.ref_name }}
-      run_id: ${{ github.run_id }}
+      run_id: ${{ github.run_id }}
+      failures: ${{ needs.test.outputs.failures }}