fix: delete the remote branch when pull request is merged

When we auto-merge a pull request we should also cleanup the existing remote branch. Issue: #31
conijnio · Feb 12, 2022 · bea4a91 · bea4a91
1 parent 9f7beb6
commit bea4a91
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/pull_request_codecommit/aws/client.py b/pull_request_codecommit/aws/client.py
@@ -119,7 +119,9 @@ def update_pull_request(
 
         return data.get("pullRequest")
 
-    def merge_pull_request(self, repository: str, pull_request_id: int) -> dict:
+    def merge_pull_request(
+        self, repository: str, pull_request_id: int, branch: str
+    ) -> dict:
         response = self.__execute(
             [
                 "codecommit",
@@ -131,5 +133,18 @@ def merge_pull_request(self, repository: str, pull_request_id: int) -> dict:
             ]
         )
         data = json.loads(response)
-
-        return data.get("pullRequest")
+        pull_request = data.get("pullRequest")
+
+        if pull_request.get("pullRequestStatus", "") == "CLOSED":
+            self.__execute(
+                [
+                    "codecommit",
+                    "delete-branch",
+                    "--repository-name",
+                    repository,
+                    "--branch-name",
+                    branch,
+                ]
+            )
+
+        return pull_request
diff --git a/pull_request_codecommit/pull_request_codecommit.py b/pull_request_codecommit/pull_request_codecommit.py
@@ -65,6 +65,7 @@ def merge(self) -> str:
         response = self.__client.merge_pull_request(
             repository=self.__repo.remote.name,
             pull_request_id=self.pull_request_id,
+            branch=self.__repo.branch,
         )
         status = response.get("pullRequestStatus", "")