<div dir="auto">Hmm, good point fangrui, I'll see what we can do to minimize that.<div dir="auto"><br></div><div dir="auto">Steve</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 11, 2020, 7:34 PM Fangrui Song <<a href="mailto:maskray@google.com">maskray@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2020-09-12, Renato Golin via llvm-dev wrote:<br>
>On Fri, 11 Sep 2020 at 23:56, Hubert Tong via llvm-dev <<br>
><a href="mailto:llvm-dev@lists.llvm.org" target="_blank" rel="noreferrer">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
>> On Fri, Sep 11, 2020 at 3:45 PM Stephen Neuendorffer via llvm-dev <<br>
>>> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank" rel="noreferrer">llvm-dev@lists.llvm.org</a>> wrote:<br>
>>><br>
>>>> Just to clarify: All the LLVM incubator repositories have "enforce<br>
>>>> linear history" enabled.  Neither "Squash and Merge" or "Rebase and Merge"<br>
>>>> results in a Merge commit in the git history.<br>
>>>><br>
>>> I am not aware of/still have not experienced a case where "Rebase and<br>
>> Merge" retains merge commits.<br>
>><br>
><br>
>It does not. We can disable the merge commit on Github settings. Plus, the<br>
>master branch history protection will stop people from merge-commit or<br>
>force-push by hand anyway.<br>
<br>
One property of "Squash and merge" is that it will add intermediate<br>
commits as bullet points (`* `). In many cases the merger does not spend<br>
more time cleaning up the description so a commit may look like:<br>
<br>
```<br>
     RFC: treat small negative λ as 0 for sqrt(::Hermitian) (#35057)<br>
<br>
     * treat small negative λ as 0 for sqrt(::Hermitian) and log(::Hermitian)<br>
<br>
     * typo<br>
<br>
     * added tests, docs; removed rtol argument for log<br>
<br>
     * don't ask for rtol so close to eps(Float64)<br>
```<br>
<br>
The typo and `added tests` messages should probably not be there. The circt commit demonstrates the exact same problem: <a href="https://github.com/llvm/circt/commit/9ab7d667f9ad5de06c7986a71ad5e07e19cd55e7" rel="noreferrer noreferrer" target="_blank">https://github.com/llvm/circt/commit/9ab7d667f9ad5de06c7986a71ad5e07e19cd55e7</a><br>
</blockquote></div>