<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 22, 2013 at 11:24 AM, Duncan Thomas <span dir="ltr"><<a href="mailto:duncan.thomas@gmail.com" target="_blank">duncan.thomas@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 22 November 2013 14:59, Ben Nemec <<a href="mailto:openstack@nemebean.com">openstack@nemebean.com</a>> wrote:<br>

<br>
> One other thought I had was to add the ability to split one Oslo sync up<br>
> into multiple commits, either one per module, or even one per Oslo commit<br>
> for some really large module changes (I'm thinking of the 1000 line db sync<br>
> someone mentioned recently).  It would be more review churn, but at least it<br>
> would keep the changes per review down to a more reasonable level.   I'm not<br>
> positive it would be beneficial, but I thought I'd mention it.<br>
<br>
</div>Cinder (often but not always me) tends to reject merges that do more<br>
that one module at a time, because it makes it far harder to review<br>
and spot problems, so some from of automation of this would be great.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">There are times when the commits are related, but in general this seems like a good practice.</div>
<div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Doug</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
--<br>
Duncan Thomas<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>