Skip to content

Commit

Permalink
fix(actions): Restore stricter checks for slack notifications
Browse files Browse the repository at this point in the history
  • Loading branch information
SokratisVidros committed May 9, 2024
1 parent c89e954 commit 9634523
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/actions/slack-notify-on-failure/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,6 @@ runs:
message_format: "{emoji} *{workflow}* {status_message} in <{repo_url}|{repo}>"
notification_title: "{workflow} is now failing!"
notify_when: "failure"
status: "failure"
status: ${{ job.status }}
env:
SLACK_WEBHOOK_URL: ${{ inputs.slackWebhookURL }}
2 changes: 1 addition & 1 deletion .github/workflows/reusable-web-e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -235,6 +235,6 @@ jobs:
path: apps/web/cypress/screenshots

- uses: ./.github/actions/slack-notify-on-failure
if: failure()
if: ${{ failure() || needs.e2e_web.result == 'failure' }}
with:
slackWebhookURL: ${{ secrets.SLACK_WEBHOOK_URL_ENG_FEED_GITHUB }}

0 comments on commit 9634523

Please sign in to comment.