[Development] Proposal: Allow contributors to +1 sanity review.

Sergio Ahumada san at sansano.inf.utfsm.cl
Wed Aug 14 21:27:29 CEST 2013


On 08/14/2013 08:22 PM, Alan Alpert wrote:
> 
> To my knowledge, there isn't really a point to having a separate
> Sanity Review field at all. I don't know why the sanity bot doesn't
> just give +/- 1, which would give effectively the same results without
> having to maintain an unusual gerrit UI. I have not seen a sanity
> review or equivalent in any other gerrit instance.

by default Gerrit has two. "Verified" and "Code Review".

in our Gerrit instance we removed Verified and added "Sanity Review".

see for example http://review.merproject.org/#change,967

actually, in our JIRA there is a suggestion to add a new (third) category.

-- 
Sergio Ahumada
san at sansano.inf.utfsm.cl



More information about the Development mailing list