RFR: 1706: Store github checks in PR unique way [v2]

Erik Joelsson erikj at openjdk.org
Thu Feb 9 23:00:35 UTC 2023


On Thu, 9 Feb 2023 22:22:51 GMT, Zhao Song <zsong at openjdk.org> wrote:

>> bots/pr/src/main/java/org/openjdk/skara/bots/pr/IntegrateCommand.java line 64:
>> 
>>> 62:         final var outdated = "the status check `" + checkName + "` has not been performed on commit %s yet";
>>> 63: 
>>> 64:         checkName = checkName.equals("jcheck") && pr.repository().forge().name().equals("GitHub") ?
>> 
>> There is just one caller of this method, which explicitly calls with `"jcheck"`. The caller should generate the new check name instead of rewriting it here.
>
> But in that way, the checkName will be "jcheck-repo#pull" and the error message showed to user would be "the status check jcheck-repo#pull is still in progress".

I think that's what we want. That's the name the check will have in the GitHub UI as well.

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

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


More information about the skara-dev mailing list