Re: Stop false review statements
From: Greg KH
Date: Sat May 16 2026 - 11:36:25 EST
On Sat, May 16, 2026 at 11:20:33AM -0400, Konstantin Ryabitsev wrote:
> On Sat, May 16, 2026 at 05:11:28AM -0700, Guenter Roeck wrote:
> > On Sat, May 16, 2026 at 10:05:02AM +0200, Krzysztof Kozlowski wrote:
> > > What the hell is that:
> > >
> > > https://lore.kernel.org/all/20260515190707.033BDC2BCB0@xxxxxxxxxxxxxxx/
> > >
> > > As a bot you CANNOT MAKE a Reviewer's statement of oversight. You are
> > > not a damn human do be able to make such statement. You are a bot, a tool.
> > >
> >
> > Where exactly do the rules say that ? I seem to miss that.
> >
> > There is a policy document about _contributions_ made by AI, but I don't
> > see the one that says that AI agents must not provide Reviewed-by: tags.
>
> >From my perspective, AI agents must NOT use the Reviewed-by tag for the
> following reasons:
>
> - We consider this a "person-trailer" and it implies agency
> - Adding yourself to a commit via a trailer is a *binding responsibility* for
> the change. A lot of tooling will cc the Reviewed-by addresses on follow-up
> messages regarding code in this commit. If the address is bogus or doesn't
> go to a developer, this is both wasteful and potentially frustrating.
I agree, any sort of "automated" tool shouldn't be adding these types of
tags.
thanks,
greg k-h