RFR: Set or update the resolved in build field when new tags arrive
Erik Helin
ehelin at openjdk.java.net
Mon Sep 14 12:47:37 UTC 2020
On Mon, 14 Sep 2020 12:10:14 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
> When the issue notifier is configured to set a build name when resolving issues, it should also update that name when
> an OpenJDK-formatted tag is seen in the repository.
Looks good, just spell out "rib" in two places
bots/notify/src/main/java/org/openjdk/skara/bots/notify/issue/BuildCompare.java line 40:
> 38: }
> 39:
> 40: // Notable values for rib are 'team', 'master', and numbered builds
Suggestion:
// Notable values for "Resolved in Build" are 'team', 'master', and numbered builds
bots/notify/src/main/java/org/openjdk/skara/bots/notify/issue/IssueNotifier.java line 296:
> 294: var optionalIssue = issueProject.issue(commitIssue.shortId());
> 295: if (optionalIssue.isEmpty()) {
> 296: log.severe("Cannot update RIB for issue " + commitIssue.id()
Suggestion:
log.severe("Cannot update "Resolved in Build" for issue " + commitIssue.id()
-------------
Marked as reviewed by ehelin (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/817
More information about the skara-dev
mailing list