On 08/15/2014 04:07 PM, Gerald Combs wrote:
On 8/15/14 12:17 PM, Jeff Morriss wrote:
Hi folks,
One thing I've noticed about the Petri-Dish flag in Gerrit is that every
time I set it to +1 I get an error saying "needs code review." Since
Chris just noticed and complained about that too (on change 3624) I
thought I'd mention it here.
Are you pressing the "Publish Comments" or "Publish and Submit" button
when this happens? We're currently using the default MaxWithBlock
setting[1] for the Code-Review label, which means that submission
requires at least one "+2" set and no "-2"s.
Ah, that's it. I was (apparently--I had to try it out to confirm)
"Publish and Submit" since, well, the other button ("Publish Comments")
is decidedly *not* what I want to do: I want to submit my +1 (and I
don't *have* any comments so I really didn't think I wanted to do that).
But I guess Submit means "Submit for merge" or some such, not "Submit my
changed value."
