-
Notifications
You must be signed in to change notification settings - Fork 1
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: Add a slack notifier workflow #18
Conversation
7895339
to
e812333
Compare
1e67971
to
5346805
Compare
Marking as draft until I solve how to keep track of timeouts. |
f303e4e
to
8e4d3eb
Compare
8e4d3eb
to
d3074ef
Compare
It should be working now. See https://github.com/CQCL/tket-json-rs/actions/runs/11520657348/job/32072548437?pr=84 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Using the new notify-slack workflow from CQCL/hugrverse-actions#18. I'll update the target branch once that is merged.
Updates the missing ops slack notifier to run daily, so we find the errors quicker. We use the new workflow from CQCL/hugrverse-actions#18 to add a one-week timeout, so we don't spam the chat. The notification step was actually broken on `main`. This PR adds a `continue-on-error` flag on the test job so it doesn't get skipped the notification when it fails.
Using the new notify-slack workflow from CQCL/hugrverse-actions#18. I'll update the target branch once that is merged.
Add a workflow based on
slackapi/slack-github-action
, that lets us rate-limit messages sent to slack to avoid spamming.