RFR: 1256: Use stream api to simplify the csr issue lookup [v3]
Magnus Ihse Bursie
ihse at openjdk.java.net
Wed Dec 1 13:46:59 UTC 2021
On Wed, 1 Dec 2021 08:00:15 GMT, Guoxiong Li <gli at openjdk.org> wrote:
>> Hi all,
>>
>> This patch adds the common methods `Issue#linksWithRelationships`, `Issue#csrLink` and `Issue#csrIssue` and refactors the code to reduce the loop nested statements by using these common methods.
>>
>> All the existing tests passed.
>>
>> Best Regards,
>> -- Guoxiong
>
> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>
> Improve the encapsulation and use the methods of the 'Optional' instead of direct null check
bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java line 139:
> 137: return Optional.empty();
> 138: }
> 139: var csr = jbsIssue.get().links().stream()
... and here as well. It's really unfortunate that we have no good framework for where to place such common functionality. :-(
-------------
PR: https://git.openjdk.java.net/skara/pull/1248
More information about the skara-dev
mailing list