Notify once

TonyCTHsu · May 23, 2024 · cd38b93 · cd38b93
1 parent d5fd359
commit cd38b93
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/.github/actions/slack/action.yml b/.github/actions/slack/action.yml
@@ -1,5 +1,7 @@
 name: Slack Notification
+
 description: 'Send slack notification'
+
 inputs:
   message:
     description: 'Message to send'
@@ -10,6 +12,7 @@ inputs:
   token:
     description: 'Slack bot token'
     required: true
+
 runs:
   using: "composite"
   steps:

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -43,14 +43,18 @@ jobs:
         DD_ENV: ci
       run: bundle exec rake spec
 
-    - if: ${{ success() }}
+  notify:
+    needs: test
+    runs-on: ubuntu-latest
+    steps:
+    - if: ${{ needs.test.result == 'success' }}
       uses: ./.github/actions/slack
       with:
         message: ":tada: Tests passed!"
         channel: ${{ vars.SLACK_CHANNEL_ID }}
         token: ${{ secrets.SLACK_BOT_TOKEN }}
 
-    - if: ${{ failure() }}
+    - if: ${{ needs.test.result == 'failure' }}
       uses: ./.github/actions/slack
       with:
         message: ":red-light: Tests failed!"