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.

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

Disable notifications and change throttle for individual checks #234

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Riley19280
Copy link
Contributor

Notifications for specific checks can be disabled all together or based on the result status, allowing for finer control over which alerts are surfaced and reducing noise from non-critical issues.

Custom throttle durations for notifications can also be set, giving the ability to manage the frequency of alerts based on the criticality of the individual checks

@freekmurze
Copy link
Member

Could you rebase against main to fix the tests?

@Riley19280 Riley19280 force-pushed the feat/conditional-notifications branch from 2f0157e to bdf9332 Compare August 1, 2024 01:34
@Riley19280
Copy link
Contributor Author

Apologies for the wait, tests are passing now 🙂

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.

2 participants