Add smarter check for alerts

CMSgov · Feb 11, 2025 · 04a1308 · 04a1308
1 parent 3d3b9e9
commit 04a1308
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/build-and-package.yml b/.github/workflows/build-and-package.yml
@@ -68,7 +68,7 @@ jobs:
             docker rmi $(docker images --filter reference="${{ secrets.ACCOUNT_ID }}${DOCKER_ECR_SUFFIX}/*:${SLIM_VERSION}" --format "{{.Repository}}:{{.Tag}}")
           fi
       # - name: Success Alert
-      #   if: ${{ success() }}
+      #   if: ${{ success() && needs.ci_checks.result == 'success' && needs.build_and_package.result == 'success' }}
       #   uses: slackapi/slack-github-action@v2.0.0
       #   with:
       #     method: chat.postMessage
@@ -82,7 +82,7 @@ jobs:
       #           mrkdown_in:
       #             - text
       # - name: Failure Alert
-      #   if: ${{ failure() }}
+      #   if: ${{ failure() || needs.ci_checks.result != 'success' || needs.build_and_package.result != 'success' }}
       #   uses: slackapi/slack-github-action@v2.0.0
       #   with:
       #     method: chat.postMessage