Skip to content

Use PR title and description for commit message#7563

Merged
alamb merged 1 commit intoapache:mainfrom
kou:commit-message
May 31, 2025
Merged

Use PR title and description for commit message#7563
alamb merged 1 commit intoapache:mainfrom
kou:commit-message

Conversation

@kou
Copy link
Copy Markdown
Member

@kou kou commented May 29, 2025

Which issue does this PR close?

Closes #7562.

Rationale for this change

We have our PR template and it has very useful information than the current default commit message:

  • The commit message of the first commit if only one commit exists in the PR
  • The commit title of the first commit as a title and list of commit titles of all commits if multiple commits exist in the PR

What changes are included in this PR?

Changed the default commit message to the PR title and description.

Are there any user-facing changes?

No.

Copy link
Copy Markdown
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @kou (and @mbrobbel)

I think we should give it a try!

@alamb alamb merged commit 66903b6 into apache:main May 31, 2025
9 checks passed
@kou kou deleted the commit-message branch May 31, 2025 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use PR title and description for commit message

3 participants