Hi Pau, On Tue, Nov 13, 2018 at 05:08:49PM +0100, Pau Espin Pedrol wrote: > I agree with implementing this kind of change, I really like the idea of > getting more people into reviewing and their vote counting so there's no > need for somebody with +2 to merge it.
ACK. > The only drawback I see is the fact that if +3 is needed, then a +2 from > some people is now not enough, which can be a pity for patches that can be > merged quickly (cosmetic, logging, etc.) or important fixes which require > quick merge. ACK. That's why I think eventually a +3 will be needed. But I'm happy to give the current proposal a (short) try. If we get a backlog of patches at "+2" stage that don't reach "+3" then we must quickly introduce +3. > Also, for some repos most people don't care bout or don't have much > knowledge about it, for instance the case os myself and osmo-gsm-tester, > where I usually +2 my patches right away or after leaving them under review > for a few hours/days. This would block my development or fix of > setup-related issues. Equally ACK here. It's frequent practise in some projects that lack a wider community. > So good for me, but we need to resolve this kind of details in the proposal > beforehand. Indeed, the +3 "privilege" should be configured from day one, with nobody enjoying it until we see a need for it. -- - Harald Welte <[email protected]> http://laforge.gnumonks.org/ ============================================================================ "Privacy in residential applications is a desirable marketing option." (ETSI EN 300 175-7 Ch. A6)
