[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Using Gerrit for code review in KDE
From:       Ben Cooksley <bcooksley () kde ! org>
Date:       2014-09-13 20:11:43
Message-ID: CA+XidOGG2CakJfXvOJmT8_y=5MfjOS8MBA7QvA7TH--Mv2vygA () mail ! gmail ! com
[Download RAW message or body]

On Sun, Sep 14, 2014 at 8:07 AM, Ivan =C4=8Cuki=C4=87 <ivan.cukic@kde.org> =
wrote:
>
>> that needs to be reverted because it's actively objectiona-
>> ble. As Ivan pointed out, few of us will ever commit any-
>> thing if we're not confident it would meet with the approval
>
> While I do agree that we have a strange and unreally awesome community th=
at
> behaves really well (and I do trust most KDE devs), I was approaching to =
this
> from the same angle as Martin.
>
> Namely, for the projects that I know the people who are actually the /cor=
e/
> team, I always wait their input before pushing something. For those that =
I
> don't know, I need to check who is in charge, and whether a 'ship it' I g=
ot
> actually has any weight behind it.
>
> +2 would show a newcommer that the review is really by someone who (1) lo=
oked
> it in-detail, and (2) by someone who actually knows what he is talking ab=
out.
> (this might sound overly strict, but I guess you know what I meant by thi=
s)

Shouldn't this be up to the reviewer to use their good judgement when
deciding whether to use +1 or +2?
If they're not the maintainer or don't know the codebase well enough,
then granting +2 would be rather unusual from a social point of view.

>
> For me, it is not about trust. But rather about providing additional
> information to the submitter. That is why I don't think that the requirem=
ent
> for the 'submit' does not need to be limitted to the maintainers/core tea=
m.
>
> Also, Kevin's idea of +1s that got more weight over time (aka the inactiv=
e-
> core-team mode) seems nice, though I don't think 1 week is the right ammo=
unt
> of time.
>
> Cheerio,
> Ivan

Thanks,
Ben

>
> --
> KDE, ivan.cukic at kde.org, http://ivan.fomentgroup.org/
> gpg key id: 850B6F76, keyserver.pgp.com
>
[prev in list] [next in list] [prev in thread] [next in thread]