<div dir="ltr"><div><div>Hi,<br><br><br></div>I like the idea  ... with complex change, it could useful for the understanding to split it into smaller changes during development.<br></div><div><br></div><div><br></div><div>
Do we need to expose such feature under git review? we could define a new subcommand? git reviewflow?<br><br><br></div><div>Cédric,<br></div><div>ZZelle@IRC<br></div><div><div><br></div></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Aug 5, 2014 at 4:49 PM, Ryan Brown <span dir="ltr"><<a href="mailto:rybrown@redhat.com" target="_blank">rybrown@redhat.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=""><br>
<br>
On 08/05/2014 09:27 AM, Sylvain Bauza wrote:<br>
><br>
> Le 05/08/2014 13:06, Ryan Brown a écrit :<br>
</div><div class="">> -1 to this as git-review default behaviour. Ideally, branches should be<br>
> identical in between Gerrit and local Git.<br>
<br>
</div>Probably not as default behaviour (people who don't want that workflow<br>
would be driven mad!), but I think enough folks would want it that it<br>
should be available as an option.<br>
<div class=""><br>
> I can understand some exceptions where developers want to work on<br>
> intermediate commits and squash them before updating Gerrit, but in that<br>
> case, I can't see why it needs to be kept locally. If a new patchset has<br>
> to be done on patch A, then the local branch can be rebased<br>
> interactively on last master, edit patch A by doing an intermediate<br>
> patch, then squash the change, and pick the later patches (B to E)<br>
><br>
> That said, I can also understand that developers work their way, and so<br>
> could dislike squashing commits, hence my proposal to have a --no-squash<br>
> option when uploading, but use with caution (for a single branch, how<br>
> many dependencies are outdated in Gerrit because developers work on<br>
> separate branches for each single commit while they could work locally<br>
> on a single branch ? I can't iimagine how often errors could happen if<br>
> we don't force by default to squash commits before sending them to Gerrit)<br>
><br>
> -Sylvain<br>
><br>
>> Cheers,<br>
><br>
><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>
<br>
</div>I am well aware this may be straying into feature creep territory, and<br>
it wouldn't be terrible if this weren't implemented.<br>
<div class="im HOEnZb"><br>
--<br>
Ryan Brown / Software Engineer, Openstack / Red Hat, Inc.<br>
<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<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>