Skip to content

Conversation

banesullivan
Copy link
Contributor

This adds https://github.com/marketplace/actions/slack-notify for notifying us on Slack when the automated data update jobs fail'

per #560 (comment)

@banesullivan banesullivan marked this pull request as ready for review March 12, 2025 02:46
Copy link
Collaborator

@willingc willingc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Member

@lwasser lwasser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@banesullivan this is great! I second @willingc approval - let's merge. Can you also please document how this is setup? Maybe that's another pr that updates the README? It's really easy to forget where the slack sync workflows live.

@lwasser lwasser merged commit 25c719a into main Mar 14, 2025
5 checks passed
@lwasser lwasser deleted the maint/ci-slack-notify branch March 14, 2025 14:41
@lwasser
Copy link
Member

lwasser commented Mar 19, 2025

@all-contributors please add @banesullivan for code, review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants