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

sendSlackMessage: don't send again if the there it increased by one f… #695

Merged
merged 1 commit into from
Oct 24, 2024

Commits on Oct 24, 2024

  1. sendSlackMessage: don't send again if the there it increased by one f…

    …inding every time
    
    use filteredMessage as a way to prevent over notification on slack.
    This would not send again any notification if there is an extra finding on a given PR,
    for the previous 50 message, or a day ago.
    
    This would debounce chaingun messages in any slack channel from the main action itself.
    
    It should not have any other side-effect.
    
    This will effectively change the hash function, so invalidate the debounce for all the previous messages.
    thypon committed Oct 24, 2024
    Configuration menu
    Copy the full SHA
    08949b5 View commit details
    Browse the repository at this point in the history