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
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ kde-community mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-community
