Hi everyone,

+1

I remember that I manually copy-pasting PR descriptions into merge commit
messages multiple times. PRs often accumulate numerous commits during
reviews, resulting in overly lengthy and less useful commit messages.
Using the PR description as the default merge commit message seems like a
more helpful and effective approach.

Best regards,
Honah

On Mon, Jan 13, 2025 at 8:15 PM Xuanwo <xua...@apache.org> wrote:

> Hi
>
> Here is my +1 non-binding.
>
> I have to change the merge commit from time to time. This setting change
> will help a lot.
>
> On Tue, Jan 14, 2025, at 11:49, Renjie Liu wrote:
>
> Hi, all:
>
> I want to propose to change the github option in iceberg-rust to use pr
> title + pr description as default git commit title and message, and github
> already has support for this
> <https://github.blog/changelog/2022-08-23-new-options-for-controlling-the-default-commit-message-when-merging-a-pull-request/>
>  .
>
> Currently when we use squash commits to merge a pr, the default git commit
> title is pr title, while git commit message is the combined message of each
> commit. In most cases pr description contains more detailed information
> describing what's happening while the commit message in each commit is
> somehow cluttered with random messages.
>
> Looking forward to hearing from you!
>
> Xuanwo
>
> https://xuanwo.io/
>
>

Reply via email to