Pull requests Merge SHA property is optional

Author: makstech

lib/Github/Api/PullRequest.php

@@ -182,7 +182,7 @@ public function merged($username, $repository, $id)
         return $this->get('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/pulls/'.rawurlencode($id).'/merge');
     }
 
-    public function merge($username, $repository, $id, $message, $sha, $mergeMethod = 'merge', $title = null)
+    public function merge($username, $repository, $id, $message, $sha = null, $mergeMethod = 'merge', $title = null)
     {
         if (is_bool($mergeMethod)) {
             $mergeMethod = $mergeMethod ? 'squash' : 'merge';
@@ -194,10 +194,12 @@ public function merge($username, $repository, $id, $message, $sha, $mergeMethod
 
         $params = [
             'commit_message' => $message,
-            'sha' => $sha,
             'merge_method' => $mergeMethod,
         ];
 
+        if (is_string($sha)) {
+            $params['sha'] = $sha;
+        }
         if (is_string($title)) {
             $params['commit_title'] = $title;
         }