RFR: 2297: CSRs for OpenJDK 8u backports not being picked up
Erik Joelsson
erikj at openjdk.org
Mon Jun 24 09:25:26 UTC 2024
On Thu, 20 Jun 2024 18:00:02 GMT, Zhao Song <zsong at openjdk.org> wrote:
> Currently, the version matching logic in `Backports::findClosetIssue` and `Backports::findIssue` differs, causing the skara bot fail to find CSRs for some issues.
>
> In this patch, I'm trying to unify the version matching logic between the two methods. I couldn't find a good way to reuse the previous methods(`Backports::matchVersion`, `Backports::matchOptPoolVersion`, `Backports::matchPoolVersion`, `Backports::matchScratchVersion`), so I changed the interface of the methods.
Marked as reviewed by erikj (Lead).
jbs/src/main/java/org/openjdk/skara/jbs/Backports.java line 172:
> 170: .filter(Backports::isNonScratchVersion)
> 171: .toList();
> 172: return nonScratch.size() == 0;
Maybe use `noneMatch`?
-------------
PR Review: https://git.openjdk.org/skara/pull/1661#pullrequestreview-2135157685
PR Review Comment: https://git.openjdk.org/skara/pull/1661#discussion_r1650655125
More information about the skara-dev
mailing list