From: Phillip Wood <phillip.wood123@gmail.com>
Date: Wed, 07 May 2025 10:26:07 GMT
Subject: Re: [RFC PATCH 1/2] rebase, am: add --reviewby option
Message-ID: <0c4721c6-9ca9-41b4-a2aa-39f521d66426@gmail.com>
In-Reply-To: <48b86c85-bc39-40ba-a2b3-67de707dd798@crinan.ddns.net>

On 07/05/2025 11:17, Phillip Wood wrote:
> Hi Li
> 
> On 07/05/2025 07:46, Li Chen wrote:
>>
>> Some projects require every commit to carry a Reviewed-by: line
>> for accountability, much like the kernel requires Signed-off-by:.
>> A first‑class option keeps that workflow “out of the box”; otherwise
>> people need to define an alias such as
>>
>> [alias]
>>      rbr = rebase --trailer "Reviewed-by: $GIT_AUTHOR_NAME 
>> <$GIT_AUTHOR_EMAIL>"
>>
>> which is functional but less convenient.
>>
>> I would appreciate your further thoughts on whether a dedicated
>> flag(--reviewby) is acceptable, or whether we should drop it and rely 
>> solely on
>> the generic --trailer interface.
> 
> I think adding support for --trailer is a good idea and if we do that we 
> don't need --reviewby. The existence and implementation of --signoff is 
> largely a historical artifact - I'm not sure we'd make the same choices 
> if we were thinking about adding it today. Different projects have 
> different requirements and I don't think it is sensible to add a new 
> option catering to the different demands of each project.

It might be worth thinking about how we could extend the trailer option 
so that it uses the committer identity if there is no value specified 
which would reduce the pain of adding things like Reviewed-by:

Best Wishes

Phillip

> I'll take a proper look at the second patch tomorrow.
> 
> Best Wishes
> 
> Phillip
> 
>> Thanks again for the review.
>>
>> Regards,
>> Li
>>
>