On 12/07/2012 04:19 PM, Antoine Musso 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
Should there be a separate label for human testing and verification (as opposed to Code Review, which could just mean "after visual review, the code looks right")?
Matt Flaschen