diff options
| author | Wladimir J. van der Laan <[email protected]> | 2017-07-11 15:39:55 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2017-07-11 15:40:08 +0200 |
| commit | ca4c545cc7e8c06bc8d9d4db1dd2d8f5fe011902 (patch) | |
| tree | a1f2ab55214fc0946f8f52c2c89d38d45d0709c1 | |
| parent | Merge #10543: Change API to estimaterawfee (diff) | |
| parent | Add PR description to merge commit in github-merge.py (diff) | |
| download | discoin-ca4c545cc7e8c06bc8d9d4db1dd2d8f5fe011902.tar.xz discoin-ca4c545cc7e8c06bc8d9d4db1dd2d8f5fe011902.zip | |
Merge #10786: Add PR description to merge commit in github-merge.py
475c08c Add PR description to merge commit in github-merge.py (Pieter Wuille)
Pull request description:
There is often some context given in PR descriptions that is missing from commits, and it may be worthwhile to retain that information in our history in git. This PR adds that information to the merge commit when created through `github-merge.py`.
We should also encourage people to provide as much information as possible in the PR commits themselves, but I believe that is an orthogonal issue. Individual commits don't need to have a description of the overall goal of a PR.
Tree-SHA512: cbae46ec24ce911744e11a07e5d51895d3acc79659db306d8124304ef8e422bba99001ee2a741b08c92a5eb39d9c3f6b723622b20d6553ca5cfa0a8de866194c
| -rwxr-xr-x | contrib/devtools/github-merge.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/contrib/devtools/github-merge.py b/contrib/devtools/github-merge.py index e9816f7d1..c664cf81f 100755 --- a/contrib/devtools/github-merge.py +++ b/contrib/devtools/github-merge.py @@ -175,6 +175,7 @@ def main(): if info is None: exit(1) title = info['title'].strip() + body = info['body'].strip() # precedence order for destination branch argument: # - command line argument # - githubmerge.branch setting @@ -229,6 +230,7 @@ def main(): firstline = 'Merge #%s' % (pull,) message = firstline + '\n\n' message += subprocess.check_output([GIT,'log','--no-merges','--topo-order','--pretty=format:%h %s (%an)',base_branch+'..'+head_branch]).decode('utf-8') + message += '\n\nPull request description:\n\n ' + body.replace('\n', '\n ') + '\n' try: subprocess.check_call([GIT,'merge','-q','--commit','--no-edit','--no-ff','-m',message.encode('utf-8'),head_branch]) except subprocess.CalledProcessError as e: |