<div dir="ltr"><div>Where I come from we always squash. More detailed history is preserved in the code review tool (which keeps a snapshot every time you bounce it back to the reviewer). Looking at my own sub-commits when I'm working on a complex feature or bug fix, they are often checkpoints with no particular significance except that the code is syntactically correct, and a common reason for doing a sub-commit is when I've got to attend to something else (e.g. a meeting).<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 2, 2015 at 6:31 AM, Barry Warsaw <span dir="ltr"><<a href="mailto:barry@python.org" target="_blank">barry@python.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Apr 02, 2015, at 12:06 PM, Jason R. Coombs wrote:<br>
<br>
>The way I see it, a squash of history or massive patch file loses history. It<br>
>loses details about the thought process of the implementer. It masks mistakes<br>
>and obscures motivations. It also masks decisions made in the merge<br>
>operation, further hiding potential problems.<br>
<br>
</span>In general I agree.  Coming from bzr, it's very rare that merges get rebased<br>
first, but bzr has a strong "mainline-of-development" view that tends to make<br>
squash-before-merge unnecessary.  diffs, bisects, logs, etc generally follow<br>
first-parents by default so you don't see all the subcommits, unless you want<br>
to, which sometimes you do.<br>
<br>
git doesn't really follow this tradition (although some commands have an<br>
option to follow first parents).  Not sure about hg.<br>
<br>
Cheers,<br>
-Barry<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
python-committers mailing list<br>
<a href="mailto:python-committers@python.org">python-committers@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-committers" target="_blank">https://mail.python.org/mailman/listinfo/python-committers</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">--Guido van Rossum (<a href="http://python.org/~guido">python.org/~guido</a>)</div>
</div>