Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BAU: Fixes deployments notifications #772

Merged
merged 1 commit into from Apr 4, 2024
Merged

BAU: Fixes deployments notifications #772

merged 1 commit into from Apr 4, 2024

Conversation

willfish
Copy link
Member

@willfish willfish commented Apr 4, 2024

Jira link

BAU

What?

I have added/removed/altered:

  • Fixes deployment notifications

Why?

I am doing this because:

  • The event to trigger a failure bypasses the usual flow which would have installed curl and jq in the alpine image. This means we don't see failed deployment notifications. Hoisting the installation fixes this

@willfish willfish merged commit 10b4198 into main Apr 4, 2024
13 of 14 checks passed
@willfish willfish deleted the BAU-fixes-deployments branch April 4, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shjohnson shjohnson shjohnson approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
2 participants
@willfish @shjohnson