Skip to content

Stable branch notification check is less strict

Steve Abrams requested to merge delivery19730-less-strict into master

What does this MR do and why?

We changed the text of the stable branch notification due to a typo. Then we tagged a new release candidate. The logic in the stable branch notification tries to find an identical comment when deciding if it should comment again.

In this case, the comment had changed slightly, so the notification was posted a 2nd time, resulting in folks receiving a nearly duplicate ping (example).

This MR updates the logic checking for an existing stable branch notification to only look for a small substring of the comment rather than the entire comment.

Related to gitlab-com/gl-infra/delivery#19730 (closed)

Author Check-list

  • [-] Has documentation been updated?
Edited by Steve Abrams

Merge request reports