[kde-community] Bikeshedding - our strength apparently *sigh*

[Kevin Krammer]

On Saturday, 2015-09-19, 21:08:27, Eike Hein wrote:
> On 09/19/2015 08:58 PM, Kevin Krammer wrote:
> > Even using a review tool in the first place is something that the
> > maintainer asks people to do.
> 
> No. We advertise ReviewBoard (and later Phab) as a general
> interface to throw code at our maintainers. "I don't look
> at ReviewBoard" is not a socially tenable position in our
> community in practice, just like "I don't look at GitHub"
> won't be*. The pressure will be to cover all places. Some
> people will say they don't want to or can't and abandon
> one for the other, and we'll have conflict over it and it
> will affect who develops for KDE and who maintains our
> products.

So, right now, a maintainer is expected to check reviewboard even if they are 
content with all holders of commit accounts to push directly.
But that as soon as there is a second option, then not checking reviewboard 
becomes acceptable?

That would then be a bonus for all maintainers who don't want to use pre-push 
reviews. They no longer have any pressure to check any review tool.

So bascially a win-win-win situation.
Maintainers who do not care about review are free to not use any, maintainers 
who want contributions from other KDE developers use Phab and maintainers who 
do not want contributions from KDE developers use whatever they feel like 
using.

If we assume that the third group even exists.
Otherwise it is a plain win-win situation, still an improvement over the the 
lose-win situation we apparently have (maintainers being socially pressured 
into using reviewboard).

Cheers,
Kevin
-- 
Kevin Krammer, KDE developer, xdg-utils developer
KDE user support, developer mentoring
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-community/attachments/20150919/13b5652c/attachment.sig>