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

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


On Thu, 15 Dec 2022 17:00:22 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 175:
> 
>> 173:                 log.info("The PR body doesn't have the CSR issue or progress, adding the csr update marker for this csr issue"
>> 174:                         + csr.id() + " for " + describe(pr));
>> 175:                 addUpdateMarker(pr);
> 
> `addUpdateMarker` isn't suitable to be called multiple times. Each call will modify the body again.

Right, I will change it, thx!

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

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


More information about the skara-dev mailing list