On Fri, Dec 7, 2012 at 4:19 PM, Antoine Musso hashar+wmf@free.fr wrote:
Le 07/12/12 04:56, Sébastien Santoro wrote
<snip> > This is very disturbing when browsing Gerrit for revisions still to > review and doesn't really give any useful information (as the > important information here would be the verified -1).
Lets open the can of worm here :)
Chad came up with another idea that would be to rename Verified to Automatic Tests and extends it to a range of -2 .. +2 where +1 will be lint ok and +2 unit tests ok. Not sure how feasible it is to tweak Gerrit this way though.
It shouldn't be hard--just two SQL queries to adjust the review labels
-Chad