On 9 July 2015 at 21:50, Sean Dague <[email protected]> wrote: > Maybe, I'm not convinced though. This should be just like config file > samples, yes you need to stay on top of them, especially as libraries > release, but it should be fine. Reviewers can also regen pretty easily > to get things to pass tests. I do merge conflict resolution like this > quite often as part of the review. > > Part of review latency is about understandability of reviews inbound. > And I think this double review bit is just going to make it worse.
There's room for us to evolve opinions on this as we go forward. Right now I see human review as a necessary evil for the interim, not a goal to hold. Perhaps that will change over time :). -Rob -- Robert Collins <[email protected]> Distinguished Technologist HP Converged Cloud __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
