<div dir="ltr"><div dir="ltr">On Fri, May 16, 2025 at 4:16 AM Volker Krause <<a href="mailto:vkrause@kde.org">vkrause@kde.org</a>> wrote:</div><div class="gmail_quote gmail_quote_container"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Donnerstag, 15. Mai 2025 16:55:01 Mitteleuropäische Sommerzeit Felix Ernst <br>
wrote:<br>
> Gitlab allows squashing commits while merging. I suggest to have the<br>
> checkbox for this ticked by default. Contributors would still be able to<br>
> untick it as they see fit.<br>
> <br>
> I think "encouraging squashing" is the better default because Gitlab has<br>
> features like the "Apply suggestion" button which produces non-descriptive<br>
> commits by default. Furthermore especially new contributors might not<br>
> manage the commit history in their merge requests. When it finally comes to<br>
> merging, the merging person might forget to tick that checkbox, or they<br>
> might have only recently been upgraded to a KDE developer account and not<br>
> know if they should tick this non-default setting, which will then lead to<br>
> a bunch of misleading/confusing/useless commits making their way to the<br>
> commit history of the master branch. Even project maintainers are unable to<br>
> rectify this then.<br></blockquote><div><br></div><div>It is possible for you - as the reviewer or MR author - to customise the message when applying suggestions (as well as batch them up into one commit)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> The way to change this default is described in<br>
> <a href="https://invent.kde.org/help/user/project/merge_requests/squash_and_merge.md" rel="noreferrer" target="_blank">https://invent.kde.org/help/user/project/merge_requests/squash_and_merge.md</a><br>
> #configure-squash-options-for-a-project. However, this setting is controlled<br>
> exclusively by KDE system administrators. In a short discussion with Ben<br>
> Cooksley he quite reasonably suggested: "I'm afraid we need a universal<br>
> consistent approach across our repositories, otherwise it will cause issues<br>
> where people get surprised by different behaviour. I'd suggest starting a<br>
> thread regarding the behaviour change to see what the general consensus<br>
> is."<br>
> (<a href="https://invent.kde.org/system/dolphin/-/merge_requests/895#note_1131215" rel="noreferrer" target="_blank">https://invent.kde.org/system/dolphin/-/merge_requests/895#note_1131215</a>)<br>
> So this is what I am now doing here.<br>
> <br>
> So, does anyone mind if we change "Squash commits when merging" to<br>
> "Encourage"?<br>
> (<a href="https://invent.kde.org/help/user/project/merge_requests/squash_and_merge.m" rel="noreferrer" target="_blank">https://invent.kde.org/help/user/project/merge_requests/squash_and_merge.m</a><br>
> d#configure-squash-options-for-a-project)<br>
<br>
Accidentally not squashing is far less destructive than accidentally squashing <br>
commits that should not have been squashed, and IME the latter is the more <br>
common case, ie. most MRs I review have a sensible history that should be <br>
preserved.<br>
<br>
I'd be totally ok with being forced to explicitly make a decision each time <br>
(not sure Gitlab has that option though), but having such a (potentially) <br>
destructive option enabled by default is IMHO way too risky.<br></blockquote><div><br></div><div>I'm afraid it does not have that capability.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Regards,<br>
Volker<br></blockquote><div><br></div><div>Cheers,</div><div>Ben </div></div></div>