Like the idea

On Mon, Aug 3, 2026 at 9:31 AM Andrei Tserakhau via dev <
[email protected]> wrote:

> +1.
>
> It's not very obvious that PR title may diverge from commit title now.
>
> Also for some PR-s there's a need to slight adjustment for scope and in
> such cases reviewer may ask to alter PR description, but syncing this into
> resulted commit message is a bit too much and people may skip this part.
>
> Best,
> Andrei
>
> пн, 3 авг. 2026 г., 18:20 Kevin Liu <[email protected]>:
>
>> Hey folks,
>>
>> I’m proposing that we change the GitHub squash-merge settings to populate
>> the commit title and body from the PR’s title and description.
>>
>> The current settings may use the branch’s commit title and messages
>> instead. This can produce unhelpful squash commits—such as “Initial
>> commit”—and cause the commit history on main to differ from the PR
>> content that was reviewed.
>>
>> Using the PR title and description would provide more consistent and
>> useful commit history, regardless of how many commits a PR contains.
>>
>> More details: https://github.com/apache/iceberg/issues/17467
>>
>> Thoughts?
>>
>> Thanks,
>> Kevin
>>
>

-- 
John Zhuge

Reply via email to