Skip to content

Code monitors: use match count, not result count in slack notification

Administrator requested to merge cc/slack-match-count into main

Created by: camdencheek

Previously, we used result count in the title, but match count in the "n more matches" trailer. This is confusing because you might get a message that says "3 new matches" but then a trailer that says "5 more matches", which doesn't make any sense. This updates it to use match counts instead of result counts.

See old example: Screen Shot 2022-03-24 at 09 48 47

Test plan

Updated tests demonstrate that this works as intended.

Merge request reports

Loading