RFR: 1691: Run Jcheck twice in CheckRun if .jcheck/conf has changed [v2]

Zhao Song zsong at openjdk.org
Thu Dec 8 17:47:47 UTC 2022


On Thu, 8 Dec 2022 15:30:51 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Now, I prefer not to change this method because I think ` CheckablePullRequest#targetHash` can help us reuse the target hash. If we add hash as an argument, we will need to call `PullRequestUtils.targetHash(localRepo)` more than once.
>
> `CheckablePullRequest#targetHash` is public so you can call it from `CheckRun`.

Oh, then I will change the method. Thanks.

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

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


More information about the skara-dev mailing list