Add support for default commit message

broady · broady · commit 533130da4cb4 · 2016-12-16T23:37:55.000-08:00
Fixes sigmavirus24#649.
diff --git a/github3/pulls.py b/github3/pulls.py
@@ -311,7 +311,7 @@ def issue_comments(self, number=-1, etag=None):
         return self._iter(int(number), url, IssueComment, etag=etag)
 
     @requires_auth
-    def merge(self, commit_message='', sha=None, squash=False):
+    def merge(self, commit_message=None, sha=None, squash=False):
         """Merge this pull request.
 
         :param str commit_message: (optional), message to be used for the
@@ -322,9 +322,11 @@ def merge(self, commit_message='', sha=None, squash=False):
             head branch.
         :returns: bool
         """
-        parameters = {'commit_message': commit_message, 'squash': squash}
+        parameters = {'squash': squash}
         if sha:
             parameters['sha'] = sha
+        if commit_message is not None:
+            parameters['commit_message'] = commit_message
         url = self._build_url('merge', base_url=self._api)
         json = self._json(self._put(url, data=dumps(parameters)), 200)
         if not json:
