[Engine-devel] Changing Gerrit -1 message

Oved Ourfalli ovedo at redhat.com
Thu Feb 21 07:46:08 UTC 2013



----- Original Message -----
> From: "Maor Lipchuk" <mlipchuk at redhat.com>
> To: engine-devel at ovirt.org
> Sent: Thursday, February 21, 2013 12:25:57 AM
> Subject: Re: [Engine-devel] Changing Gerrit -1 message
> 
> I tend to agree as well regarding the -1 message, although I think
> that
> since -2 blocks your change from, merging it, it should still be
> strict.
> Perhaps we could consider adding one more negative value?
> 
In my opinion, adding another negative value isn't so friendly... we should be more positive and not negative :-)
I liked the "-1: Please review my comments" proposed below.
I think the current "-2: do not submit" is pretty polite .... If you guys think it requires a change, then I'd change it to: "-2: do not submit. Please review my comments."

Oved
> Regards,
> Maor
> 
> On 02/20/2013 11:59 AM, Antoni Segura Puimedon wrote:
> > I like most of these proposals. It'll make gerrit friendlier :-)
> > 
> > ----- Original Message -----
> >> From: "Itamar Heim" <iheim at redhat.com>
> >> To: "Laszlo Hornyak" <lhornyak at redhat.com>
> >> Cc: engine-devel at ovirt.org
> >> Sent: Wednesday, February 20, 2013 9:32:53 AM
> >> Subject: Re: [Engine-devel] Changing Gerrit -1 message
> >>
> >> On 19/02/2013 12:06, Laszlo Hornyak wrote:
> >>> Hi,
> >>>
> >>> I agree with that.
> >>>
> >>> for the - messages this in my opinion would be both more clear
> >>> and
> >>> friendly:
> >>> -1: In my opinion it needs work.
> >>
> >>
> >> how about
> >> "-1: Please review my comments"
> >>
> >>> -2: I disagree.
> >>
> >> "-2: Please reconsider"
> >>
> >>>
> >>> ----- Original Message -----
> >>>> From: "Ofer Schreiber" <oschreib at redhat.com>
> >>>> To: engine-devel at ovirt.org
> >>>> Sent: Tuesday, February 19, 2013 10:51:15 AM
> >>>> Subject: [Engine-devel] Changing Gerrit -1 message
> >>>>
> >>>> I feel that the current "-1 I would prefer that you didn't
> >>>> submit
> >>>> this" message in Gerrit is pretty rude, as usually those -1
> >>>> reviews
> >>>> are just small fix-ups in the code itself.
> >>>>
> >>>> Any thoughts about a more suitable "-1" message?
> >>>> I thought about "-1 Please fix your code" or something similar.
> >>>>
> >>>> Thanks,
> >>>> Ofer Schreiber
> >>>>
> >>>> _______________________________________________
> >>>> Engine-devel mailing list
> >>>> Engine-devel at ovirt.org
> >>>> http://lists.ovirt.org/mailman/listinfo/engine-devel
> >>>>
> >>> _______________________________________________
> >>> Engine-devel mailing list
> >>> Engine-devel at ovirt.org
> >>> http://lists.ovirt.org/mailman/listinfo/engine-devel
> >>>
> >>
> >> _______________________________________________
> >> Engine-devel mailing list
> >> Engine-devel at ovirt.org
> >> http://lists.ovirt.org/mailman/listinfo/engine-devel
> >>
> > _______________________________________________
> > Engine-devel mailing list
> > Engine-devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/engine-devel
> > 
> 
> 
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
> 



More information about the Engine-devel mailing list