RFR: 1714: CSR detection only happens on the title bug of a PR

Zhao Song zsong at openjdk.org
Thu Dec 15 18:50:35 UTC 2022


On Thu, 15 Dec 2022 17:48:06 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Currently, our bots only detect CSR issues that are directly linked to the main issue. 
>> 
>> With this patch, our bots will be able to detect CSR issues that are linked to all of the issues that this PR solves. Additionally, all detected CSR issues will affect the integration blockers.
>> 
>> The `/csr needed` command will allow you to create a CSR issue that is linked to any of the issues solved by this PR. The `/csr unneeded` command will be blocked if any of the issues associated with this PR have an active CSR issue.
>
> bots/csr/src/main/java/org/openjdk/skara/bots/csr/PullRequestWorkItem.java line 136:
> 
>> 134:         var issues = new ArrayList<org.openjdk.skara.vcs.openjdk.Issue>();
>> 135:         mainIssue.ifPresent(issues::add);
>> 136:         issues.addAll(SolvesTracker.currentSolved(pr.repository().forge().currentUser(), pr.comments()));
> 
> The `SolvesTracker` in the pr bot shouldn't be reachable from this module. If you need it in multiple bots, then please move it to bots.common.

Thanks, fixed it.

-------------

PR: https://git.openjdk.org/skara/pull/1443


More information about the skara-dev mailing list