Proposal: make squash-merging the default behavior for gitlab MRs

Johan Ouwerkerk jm.ouwerkerk at gmail.com
Sat Oct 3 11:36:26 BST 2020


On Sat, Oct 3, 2020 at 10:15 AM Boudewijn Rempt <boud at valdyas.org> wrote:
>
> On Friday, 2 October 2020 19:39:37 CEST Nate Graham wrote:
>
> > Thoughts?
>
> Like others have said, please, no. Squashed commits are the worst things to have in a git history. They make it hard to use git blame, they make it hard to read the history... And the whole argument about making life easier for drive-by contributors and newbies and people not familiar with git isn't to the point anyway: those people will make a fork, do their work, make a merge request, get that reviewed, and then the maintainer does the merge.
>

This last bit is worth emphasising strongly. The whole thing is only
at issue because in KDE we assume every developer can land their
changes anywhere and rely on an honour system to enforce that it is
properly reviewed (with the exception of sysadmin stuff). But many
drive-by contributors and newbies or even seasoned contributors won't
be deterred at all from not having commit access to the main repo.
Additionally I suspect not many people will object to a maintainer
taking their changes but landing them in different commits or a
slightly different form either.

Regards,

- Johan Ouwerkerk


More information about the kde-devel mailing list