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

Simon Marchi via Gdb <[email protected]>
Newsgroups gmane.comp.gdb.devel
Message-ID <[email protected]>

On 2022-09-21 07:04, Bruno Larsen via Gdb wrote:
> TL;DR: I want to introduce the usage of 3 new review tags to the GDB patch review workflow. They are: Reviewed-by, Approved-by and Tested-by.

Hi Bruno,

I completely agree with the proposal.  I really like the fact that it
makes communication less ambiguous.  Following some process (or changing
the process) can feel a bit heavy for long-timers, but I think it makes
things much clearer for newcomers.

Assuming we will go through with this proposal, it will need to be
documented on the wiki so we can easily refer people to the procedure.
Probably the ContributionChecklist page?

  https://sourceware.org/gdb/wiki/ContributionChecklist

Will you be able to take care of this when needed (do you have write
access to the wiki)?

In the mean time, message to others: please let us know if you agree
with this, it's difficult to know we have the support of the community
if everybody silently agrees!

Simon
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.