- Notifications
You must be signed in to change notification settings - Fork927
chore: only notify about CI failure onmain
ifrequired
job fails#16114
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Signed-off-by: Danny Kopping <danny@coder.com>
ethanndickson commentedJan 13, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
coder/.github/workflows/ci.yaml Line 1283 ine53b675
From your earlier screenshot, I believe this will always be notify-slack-on-failure . Idk how easy fixing it is - can probably just remove for now, the notif thatsomething has failed should be good enough! |
Haha Ijust noticed that now too. Looking into it. |
Signed-off-by: Danny Kopping <danny@coder.com>
22236f2
intomainUh oh!
There was an error while loading.Please reload this page.
This should be the last PR to get this working
Looks like the
nightly-gauntlet
is working as expected, and this is a clone of that.@ethanndickson@matifali sorry fellas, last review from me I'm hoping