RFR: 1937: Re-evaulate all PRs when .jcheck/conf changes [v4]
Erik Joelsson
erikj at openjdk.org
Wed Jun 14 17:57:09 UTC 2023
On Wed, 14 Jun 2023 16:09:33 GMT, Zhao Song <zsong at openjdk.org> wrote:
>> As Erik said in the issue: "When we bump the version in .jcheck/conf from N to N+1 in the jdk master branch, PRs with CSRs are often not updated to reflect this."
>>
>> This root cause of this is that changing of .jcheck/conf in target ref of the pr will not be able to trigger checkWorkItem for this pr.
>>
>> In this patch,
>> 1. The prBot will monitor the changes to the .jcheck/conf file in any branch(except for PR branches).
>> 2. If the prBot detects an update to the .jcheck/conf file in a branch, it will trigger a checkWorkItem for all open PRs whose target ref is the branch where the .jcheck/conf file was updated.
>> 3. To solve the problem related with bot restarting, the contents of the .jcheck/conf file in the target branch of a PR will be included in the metadata hash.
>
> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>
> add line breaks back
Marked as reviewed by erikj (Lead).
-------------
PR Review: https://git.openjdk.org/skara/pull/1531#pullrequestreview-1480015224
More information about the skara-dev
mailing list