[External] : Re: The bot doesn't add commit comment to the JMH issues
erik.joelsson at oracle.com
erik.joelsson at oracle.com
Thu Sep 22 22:19:45 UTC 2022
On 9/22/22 11:55, Aleksey Shipilev wrote:
> On 9/22/22 15:32, Guoxiong Li wrote:
>> Actually I can't confirm whether it is necessary. So I post this
>> email to discuss with you.
>> If Aleksey Shipilev, the maintainer of the JMH, confirm we need to do
>> that,
>> I will file the SKARA admin task for this.
>
> Yes, it would be nice we could handle this better.
>
> One solution is allowing bots to close the issue automatically after a
> manual merge.
>
> But, stepping back a bit, the root cause for these projects is that
> when the repo owner (me in case of JMH) needs to /integrate
> "directly", it is not possible to do so, because bots require reviews,
> and GitHub does not accept self-reviews.
>
> Allowing zero reviews is a no-go, because it allows committer to
> integrate without review. If it was possible to have the "whitelist"
> of users who can integrate directly, then people like me can just add
> themselves to it and use the regular /integrate. Maybe there is such
> an option already, and I just missed it.
>
As long as the commit message follows the standard, Skara can react to
commits, just like it does in other repos. We just need to change the
configuration. In mainline JDK (and most other repos), it's not the
integration of a PR that triggers Skara to notify JBS, but the
appearance of a new commit in a branch.
/Erik
More information about the skara-dev
mailing list