Re: [MAINTAINERS SUMMIT] Maintainer burnout

Steven Rostedt
2023-08-29T12:54:09Z

On Mon, 21 Aug 2023 21:23:18 +0200
Vegard Nossum <vegard.nossum@oracle.com> wrote:

> Can we maybe adjust the standards of the Reviewed-by: tag to mean that
> the person has looked at the patch (or an earlier version of it) and
> provided comments that show they actually put some effort into it?
> For example, if a patch is changing a function (as patches often do),
> the reviewer should add a line saying "error paths in foo() lgtm"
> and not just tack on their Reviewed-by: line.
>
> This adjustment would make it harder to just slap a Reviewed-by: on a
> patch, but it would also make it easier to get your name in the
> changelog provided that you actually put the effort in.

Note, when I can't do a real review, I just slap an Acked-by tag to it.
That to me means LGTM, but I haven't looked deep into it.

-- Steve