On 12/27/2012 01:18 PM, Juliusz Gonera wrote: > * What is the difference between Verified and Code Review? When would I > put +1 in one of them but -1 in the other?
I'm relatively new, but this is my understanding. Verified means you actually tested it. Code Review means it "looks good" > * What is the difference between +1 and +2, especially in Verified? I think just how certain you are. > * Why do we even have +2? +1 means that someone else must approve. What > does +2 mean? No one else has to approve but I'm not merging anyway, why? +2 means it's ready to merge. In core, this will cause unit tests to run, and if they pass, it will automatically merge. I don't know of any reason (in any code) to vote CR +2 if you don't think it's ready to merge. Matt Flaschen _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l