Using Gerrit for code review in KDE

Ivan Čukić ivan.cukic at kde.org
Sat Sep 13 21:07:04 BST 2014


> 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 that 
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 /core/ 
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 got 
actually has any weight behind it.

+2 would show a newcommer that the review is really by someone who (1) looked 
it in-detail, and (2) by someone who actually knows what he is talking about. 
(this might sound overly strict, but I guess you know what I meant by this)

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 requirement 
for the 'submit' does not need to be limitted to the maintainers/core team.

Also, Kevin's idea of +1s that got more weight over time (aka the inactive-
core-team mode) seems nice, though I don't think 1 week is the right ammount 
of time.

Cheerio,
Ivan

-- 
KDE, ivan.cukic at kde.org, http://ivan.fomentgroup.org/
gpg key id: 850B6F76, keyserver.pgp.com





More information about the kde-core-devel mailing list