Update last commit message when merging with links to the PR and its title

dlang / dlang-bot

dlang-bot for automated bugzilla, github, and trello references

https://bot.dlang.io/

MIT License

22 stars 14 forks source link

Update last commit message when merging with links to the PR and its title #262

Open PetarKirov opened 3 years ago

PetarKirov commented 3 years ago

When dlang-bot does auto-merge-rebase or auto-merge-squash, it should update the message of the commit that the branch head points to with information like:

Merged-on-behalf-of: <reviewer>
Pull-Request: <url>
Pull-Request-Title:<text>
Pull-Request-Description: <text>

CyberShadow commented 3 years ago

Pull-Request:

The pull request number should always be on the first line. (This wasn't the case when the other merge methods were first introduced.)

Pull-Request-Title:

The first line should already be the pull request title.

Pull-Request-Description:

The description often contains information for reviewers, and thus usually doesn't need to be included in Git history.

Have I missed anything?