On Mar 30, 2012, at 2:24 AM, Tim Starling wrote:
On 29/03/12 00:10, Chad wrote:
Hi everyone,
There's been some comments that the phrasing for a -1 vote in
Gerrit ("I'd prefer that you didn't submit this") is kind of personal
and we can do better.
I did some testing and this is totally configurable :) It won't change
for old comments that were already submitted, but we can pick
some nicer wording going forward.
I really don't have any good suggestions for this, so I'm opening
this up to the list for a bit of good old fashioned bikeshedding.
I don't really want Gerrit putting words into my mouth regardless of
how nice they sound. There will always be cases where the phrase is
inappropriate and offputting, regardless of which one you choose.
How about "Set code review score to -1"? Then a more personal message
can be typed by the human doing the review.
-- Tim Starling
I couldn't agree more. So far all proposal make implications that sometimes
simply aren't appropriate. Either they leave no room for fixing it ("Don't
submit it"), or are too much foccused on fixing something small, but
implying the overal intention is wanted ("Needs improvement") etc. etc.
Just say what you want to say in a comment, the numbers don't add up and
are only a brief summary (also note that you can submit a different score
at anytime and it will replace your previous score).
Can we just set it to an empty string and let the numbers and hand-written
comment speak for themselves?
-- Krinkle
On Mar 29, 2012, at 11:23 PM, Krinkle wrote:
+1 for "There is a problem with this
patchset"
(without ", please improve").
I think that keeps it more neutral without saying anything the user doesn't
intend to say. It also keeps free ambiguity in the intention (to be disambiguated
in a comment) between 'wontfix' and 'fixme'.
-- Krinkle