fix: honor parameter value passed

bitte-ein-bit · bitte-ein-bit · commit c2f8f0e7db95 · 2021-02-15T13:55:14.000+01:00
Gitlab allows setting the defaults for MR to delete the source. Also
the inline help of the CLI suggest that a boolean is expected, but no
matter what value you set, it will always delete.
diff --git a/gitlab/v4/objects/merge_requests.py b/gitlab/v4/objects/merge_requests.py
@@ -312,8 +312,8 @@ def merge(
         data = {}
         if merge_commit_message:
             data["merge_commit_message"] = merge_commit_message
-        if should_remove_source_branch:
-            data["should_remove_source_branch"] = True
+        if should_remove_source_branch is not None:
+            data["should_remove_source_branch"] = should_remove_source_branch
         if merge_when_pipeline_succeeds:
             data["merge_when_pipeline_succeeds"] = True
 
