Re: Proposal: Add review tags to patch review workflow.

Eli Zaretskii via Gdb <[email protected]>
Newsgroups gmane.comp.gdb.devel
Message-ID <[email protected]>
> Date: Mon, 10 Oct 2022 12:11:46 +0200
> Cc: [email protected], [email protected]
> From: Bruno Larsen <[email protected]>
> 
> > I'm not clear what I should do when I approve just part of a patch.
> > It is frequently the case that a patch includes both code and
> > documentation, and I'm approving just the documentation part(s).  Is
> > that item 1 or item 2? or something else?
> >
> It's a bit up to you, if I'm honest. I would default to telling you to 
> use Reviewed-by, to avoid confusion, but if you want to say that the 
> "documentation parts are Approved-by", I am fine with it.
> 
> Just let me know if you decide to go with the second, so I can mention 
> in the wiki something like "make sure all of your patch is approved 
> before pushing".

I don't mind either way.  This whole thing is a service to others, so
I'll do whatever people prefer.  Let me just point out that my
situation is not too unique: several other maintainers can approve
only parts of patches.

Thanks.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.