diff --git a/bots/pr/src/main/java/org/openjdk/skara/bots/pr/CommitCommandWorkItem.java b/bots/pr/src/main/java/org/openjdk/skara/bots/pr/CommitCommandWorkItem.java index 578d50164..6c2dd2c3d 100644 --- a/bots/pr/src/main/java/org/openjdk/skara/bots/pr/CommitCommandWorkItem.java +++ b/bots/pr/src/main/java/org/openjdk/skara/bots/pr/CommitCommandWorkItem.java @@ -87,7 +87,7 @@ public boolean concurrentWith(WorkItem other) { if (!bot.repo().webUrl().equals(otherItem.bot.repo().webUrl())) { return true; } - if (!commitComment.commit().equals(otherItem.commitComment.commit())) { + if (!commitComment.id().equals(otherItem.commitComment.id())) { return true; } return false; diff --git a/bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestBot.java b/bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestBot.java index dcfc0db9f..05e827311 100644 --- a/bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestBot.java +++ b/bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestBot.java @@ -25,7 +25,6 @@ import org.openjdk.skara.bot.*; import org.openjdk.skara.census.Contributor; import org.openjdk.skara.forge.*; -import org.openjdk.skara.host.HostUser; import org.openjdk.skara.issuetracker.*; import org.openjdk.skara.json.JSONValue; @@ -34,8 +33,7 @@ import java.time.*; import java.util.*; import java.util.concurrent.ConcurrentHashMap; -import java.util.logging.Logger; -import java.util.logging.Level; +import java.util.logging.*; import java.util.regex.Pattern; import java.util.stream.Collectors; @@ -192,15 +190,10 @@ public List getPeriodicItems() { prs = remoteRepo.pullRequests(ZonedDateTime.now().minus(Duration.ofDays(1))); } - List commitComments = List.of(); - try { - commitComments = remoteRepo.recentCommitComments() + var commitComments = remoteRepo.recentCommitComments() .stream() - .filter(cc -> !processedCommitComments.contains(cc.id())) + .filter(cc -> !processedCommitComments.containsKey(cc.id())) .collect(Collectors.toList()); - } catch (Throwable e) { - log.log(Level.SEVERE, "Could not fetch commit comments for " + remoteRepo.name(), e); - } return getWorkItems(prs, commitComments); }