[Mesa-dev] Workflow Proposal
eero.t.tamminen at intel.com
Thu Oct 7 08:38:32 UTC 2021
On 6.10.2021 23.00, Jordan Justen wrote:
> Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl> writes:
>> On Wed, Oct 6, 2021 at 8:49 PM Jordan Justen <jordan.l.justen at intel.com> wrote:
>>> I guess I missed where it was suggested that Marge should remove
>>> Reviewed-by tags. I don't think Marge should ever remove something from
>>> the commit message.
>> AFAIU this is upstream Marge behavior. Once you enable the
>> Approval->Rb tag conversion it removes existing Rb tags. Hence why we
>> don't have the conversion enabled.
> Ah, I guess it is documented for --add-reviewers here:
> "All existing Reviewed-by: trailers on commits in the branch will be
This sounds horrible from the point of view of trying to track down
somebody who knows about what's & why's of some old commit that is later
on found to cause issues...
For whom those extra Rb tags are a problem and why?
> I hope we would wait for Marge to add a --add-approvers switch which
> would leave Reviewed-by tags alone, but add Approved-by tags.
More information about the mesa-dev