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