Gitlab experiment wrapup

[Alexandre Julliard]

Huw Davies <huw at codeweavers.com> writes:

> On Wed, Jun 15, 2022 at 04:06:03PM +0200, Alexandre Julliard wrote:
>> Huw Davies <huw at codeweavers.com> writes:
>> 
>> > On Tue, Jun 14, 2022 at 10:01:23PM +0200, Alexandre Julliard wrote:
>> >> Another option would be to require the submitter to explicitly approve
>> >> their own MR if it contains a patch written by someone else; that would
>> >> be similar to a reviewer pushing fixups and then approving the MR.  It
>> >> would require some mechanism to detect missing approvals though, so it
>> >> may not be worth the trouble.
>> >
>> > Could the (yet to be written) auto-add-reviewer script check whether
>> > the submitter and author are different and in that case auto add the
>> > submitter as a reviewer (as well as any other appropriate reviewers)?
>> > Then the approval check just becomes the same as for regular
>> > reviewers.
>> 
>> Yes, that would be a way of doing it, if we decide that we want that
>> info in the git notes. Do you feel that it would be useful to have it
>> there?
>
> If you're sending in someone else's work then you're essentially
> acting as a reviewer, so it seems logical to add an Approved-by tag in
> the notes.

Makes sense.

> Of course I'm not the one writing the bot, so if it's a
> pain then perhaps it's not worth the hassle as this doesn't happen
> that often.

Well, I volunteered Jer to write the bot, so I'm sure he will enjoy a
little challenge ;-)

-- 
Alexandre Julliard
julliard at winehq.org