[SciPy-Dev] On the topic of separate merge commits

[Stefan van der Walt]

On Wed, Aug 25, 2021, at 07:04, Eric Larson wrote:
> Some contributors are very good about structuring/restructuring their PRs this way, but it seems to be the (rare) exception and not the rule.

This is in line with my experience.  If you have developers who are willing to craft their patches very carefully, a merge strategy makes sense.  Otherwise, since PRs are meant to be focused in scope already, squashing works well in practice.

Github also implicitly encourages an append-to-PR workflow.  They have "view only what has changed" when you only append patches, and if I'm not mistaken cannot always track comments across rebase.

Perhaps most importantly, I think it is unreasonable to expect newcomers to rebase their work when merge is so much easier.  So, for those PRs we should squash anyway.

Eric, I agree with what you wrote.  For `git bisect` the `--first-parent` flag could be useful in only evaluating commits on the root of the commit tree.

Stéfan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.python.org/pipermail/scipy-dev/attachments/20210825/6f64c4ea/attachment.html>