Skip to content
Permalink
Browse files
666: Integrated message delivered to wrong thread
Reviewed-by: ehelin
  • Loading branch information
rwestberg committed Mar 10, 2021
1 parent 10ee68e commit 591fb2c55991988e35c1baa90142d8d2e77e8d79
Showing with 0 additions and 4 deletions.
  1. +0 −4 bots/mlbridge/src/main/java/org/openjdk/skara/bots/mlbridge/ReviewArchive.java
@@ -158,10 +158,6 @@ private List<ArchiveItem> generateArchiveItems(List<Email> sentEmails, Repositor
} else {
throw new RuntimeException("PR " + pr.webUrl() + " has integrated label but no integration comment");
}
} else if (localRepo.isAncestor(pr.headHash(), PullRequestUtils.targetHash(pr, localRepo))) {
var commit = localRepo.lookup(pr.headHash());
var reply = ArchiveItem.integratedNotice(pr, localRepo, commit.orElseThrow(), hostUserToEmailAuthor, parent, subjectPrefix);
generated.add(reply);
} else if (threadPrefix.equals("RFR")) {
var reply = ArchiveItem.closedNotice(pr, hostUserToEmailAuthor, parent, subjectPrefix);
generated.add(reply);

1 comment on commit 591fb2c

@openjdk-notifier
Copy link

@openjdk-notifier openjdk-notifier bot commented on 591fb2c Mar 10, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.