RFR: 2326: Don't send the ready for sponsor comment to mailing list if the pr is closed

David Holmes dholmes at openjdk.org
Thu Jul 11 23:17:29 UTC 2024


On Thu, 11 Jul 2024 22:13:57 GMT, Zhao Song <zsong at openjdk.org> wrote:

> After we deployed [SKARA-2302](https://bugs.openjdk.org/browse/SKARA-2302) and [SKARA-2322](https://bugs.openjdk.org/browse/SKARA-2322), some users complained about receiving numerous "ready for sponsor" emails from closed PRs.
> We should prevent the bot from retroactively sending out these emails.

bots/mlbridge/src/main/java/org/openjdk/skara/bots/mlbridge/ArchiveWorkItem.java line 156:

> 154:         if (bot.ignoredUsers().contains(author.username())) {
> 155:             if (pr.isOpen()) {
> 156:                 return !PullRequestConstants.READY_FOR_SPONSOR_MARKER_PATTERN.matcher(body).find() && pr.isOpen();

You already know `pr.IsOpen()` is true here.

-------------

PR Review Comment: https://git.openjdk.org/skara/pull/1676#discussion_r1674811729


More information about the skara-dev mailing list