+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
>

Reply via email to