Sean Dague said on Mon, Sep 08, 2014 at 09:22:56AM -0400: > > On 09/08/2014 05:17 AM, Steven Hardy wrote: > >> I think this may be a sensible move, but only if it's used primarily to > >> land the less complex/risky patches more quickly. > > 2 +2 has been part of OpenStack culture for a long time, and there is a > good reason for it, it really does keep bugs out. > > It should also be clear that the subject of this email really should > have been "merging code faster", because nothing in here doubles the > review bandwidth, it just provides us with less review coverage.
For these reasons, I'm also wary of changing this in general. Sometimes I yell in IRC if I +2 something important, this shortens loop time as hopefully I already understand the patch and can answer questions. Single-approvals for small + simple changes could be worth trying. Perhaps also for large + simple changes like whitespace fixes. Alexis -- Nova Engineer, HP Cloud. AKA lealexis, lxsli. _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev