> tl;dr I think we should drop our 'needs 2x+2 to land' rule and instead > use 'needs 1x+2'. We can ease up a large chunk of pressure on our > review bottleneck, with the only significant negative being that core > reviewers may see less of the code going into the system - but they > can always read more to stay in shape if thats an issue :)
I'm going to make a tangential but somewhat related suggestion. Instead of reducing the number of +2s required, what would happen if we gave our more trusted but non-core reviewers the ability to +2, but leave +A in the hands of the existing cores? That way, their reviews can be counted by the core reviewers. With this change in policy, you still need two +2s, but you have more people that can +2, and you only need one of our limited number of cores to review. -- Kevin L. Mitchell <[email protected]> Rackspace _______________________________________________ OpenStack-dev mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
