Merge pull request #132 from Nerixyz/fix/empty-commit

fix: support empty commits/PRs
ZedThree · Jul 19, 2024 · 0eccb1c · 0eccb1c
2 parents c373d72 + 991b995
commit 0eccb1c
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/post/clang_tidy_review/clang_tidy_review/__init__.py b/post/clang_tidy_review/clang_tidy_review/__init__.py
@@ -286,14 +286,17 @@ def pull_request(self):
             self._pull_request = self.repo.get_pull(int(self.pr_number))
         return self._pull_request
 
-    def get_pr_diff(self) -> List[unidiff.PatchSet]:
+    def get_pr_diff(self) -> List[unidiff.PatchedFile]:
         """Download the PR diff, return a list of PatchedFile"""
 
         _, data = self.repo._requester.requestJsonAndCheck(
             "GET",
             self.pull_request.url,
             headers={"Accept": f"application/vnd.github.{'v3.diff'}"},
         )
+        if not data:
+            return []
+
         diffs = data["data"]
 
         # PatchSet is the easiest way to construct what we want, but the