Skip to content

Handle multiline commit messages when creating PR#149426

Merged
joaomoreno merged 3 commits intomicrosoft:mainfrom
rojvv:main
May 18, 2022
Merged

Handle multiline commit messages when creating PR#149426
joaomoreno merged 3 commits intomicrosoft:mainfrom
rojvv:main

Conversation

@rojvv
Copy link
Copy Markdown
Contributor

@rojvv rojvv commented May 13, 2022

  • This makes the body of the pull request non-empty when the commit message consists of more than one line.
  • The GitHub web client fills the body the same way, too, when you create a pull request.
  • Some commit messages consist of more than one line, like this one, which is giving more detail about the commit and can be used as a shortcut to not write the same text when opening a pull request and on GitHub.
  • Without these changes, the pull request title will be messed up if the commit message is similar to this.

rojvv added 2 commits May 13, 2022 00:56
- This makes the body of the pull request non-empty when the commit message consists more than one line.
- The GitHub web client fills the body the same way, too, when you create a pull request.
- Some commit messages consist more than one line, like this one, which is giving more detail about the commit, and can be used as a shortcut to not write the same text when opening a pull request and on GitHub.
- Without these changes, the pull request title will be messed up if the commit message is similar to this.
@joaomoreno joaomoreno added this to the May 2022 milestone May 18, 2022
@joaomoreno joaomoreno added the github Github extension label May 18, 2022
@joaomoreno joaomoreno merged commit 64e0959 into microsoft:main May 18, 2022
@joaomoreno
Copy link
Copy Markdown
Member

Thanks!

@github-actions github-actions bot locked and limited conversation to collaborators Jul 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

github Github extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants