Fixes

Author: CTY-git

patchwork/common/client/scm.py

@@ -409,16 +409,12 @@ def reset_comments(self) -> None:
                     self.git_client.delete_comment(repository_id=self._pr.repository.id, pull_request_id=self.id, thread_id=thread.id, comment_id=comment_id, project=self._pr.repository.project.id)
 
     def texts(self) -> PullRequestTexts:
-        self.git_client.get_commit_diffs(
-            repository_id=self._pr.repository.id,
-            project=self._pr.repository.project.id
-        )
-
         target_branch = self._pr.last_merge_source_commit.commit_id
         feature_branch = self._pr.last_merge_target_commit.commit_id
 
         repo = git.Repo(path=Path.cwd(), search_parent_directories=True)
-        repo.git.fetch()
+        for remote in repo.remotes:
+            remote.fetch()
         target_commit = repo.commit(target_branch)
         feature_commit = repo.commit(feature_branch)
 

patchwork/steps/CreatePRComment/CreatePRComment.py

@@ -1,4 +1,4 @@
-from patchwork.common.client.scm import GithubClient, GitlabClient
+from patchwork.common.client.scm import GithubClient, GitlabClient, AzureDevopsClient
 from patchwork.logger import logger
 from patchwork.step import Step, StepStatus
 
@@ -15,6 +15,8 @@ def __init__(self, inputs: dict):
             self.scm_client = GithubClient(inputs["github_api_key"])
         elif "gitlab_api_key" in inputs.keys():
             self.scm_client = GitlabClient(inputs["gitlab_api_key"])
+        elif "azuredevops_api_key" in inputs.keys():
+            self.scm_client = AzureDevopsClient(inputs["azuredevops_api_key"])
         else:
             raise ValueError(f'Missing required input data: "github_api_key" or "gitlab_api_key"')
 

patchwork/steps/CreatePRComment/typed.py

@@ -11,8 +11,9 @@ class __CreatePRCommentRequiredInputs(TypedDict):
 class CreatePRCommentInputs(__CreatePRCommentRequiredInputs, total=False):
     noisy_comments: Annotated[bool, StepTypeConfig(is_config=True)]
     scm_url: Annotated[str, StepTypeConfig(is_config=True)]
-    gitlab_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["github_api_key"])]
-    github_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["gitlab_api_key"])]
+    gitlab_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["github_api_key", "azuredevops_api_key"])]
+    github_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["gitlab_api_key", "azuredevops_api_key"])]
+    azuredevops_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["gitlab_api_key", "github_api_key"])]
 
 
 class CreatePRCommentOutputs(TypedDict):

patchwork/steps/ReadPRDiffs/ReadPRDiffs.py

@@ -1,6 +1,6 @@
 from typing_extensions import List
 
-from patchwork.common.client.scm import GithubClient, GitlabClient
+from patchwork.common.client.scm import GithubClient, GitlabClient, AzureDevopsClient
 from patchwork.step import Step
 from patchwork.steps.ReadPRDiffs.typed import ReadPRDiffsInputs, ReadPRDiffsOutputs
 
@@ -26,17 +26,16 @@ def filter_by_extension(file, extensions):
 
 
 class ReadPRDiffs(Step, input_class=ReadPRDiffsInputs, output_class=ReadPRDiffsOutputs):
-    required_keys = {"pr_url"}
 
     def __init__(self, inputs: dict):
         super().__init__(inputs)
-        if not all(key in inputs.keys() for key in self.required_keys):
-            raise ValueError(f'Missing required data: "{self.required_keys}"')
 
         if "github_api_key" in inputs.keys():
             self.scm_client = GithubClient(inputs["github_api_key"])
         elif "gitlab_api_key" in inputs.keys():
             self.scm_client = GitlabClient(inputs["gitlab_api_key"])
+        elif "azuredevops_api_key" in inputs.keys():
+            self.scm_client = AzureDevopsClient(inputs["azuredevops_api_key"])
         else:
             raise ValueError(f'Missing required input data: "github_api_key" or "gitlab_api_key"')
 

patchwork/steps/ReadPRDiffs/typed.py

@@ -10,8 +10,9 @@ class __ReadPRDiffsRequiredInputs(TypedDict):
 
 class ReadPRDiffsInputs(__ReadPRDiffsRequiredInputs, total=False):
     scm_url: Annotated[str, StepTypeConfig(is_config=True)]
-    gitlab_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["github_api_key"])]
-    github_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["gitlab_api_key"])]
+    gitlab_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["github_api_key", "azuredevops_api_key"])]
+    github_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["gitlab_api_key", "azuredevops_api_key"])]
+    azuredevops_api_key: Annotated[str, StepTypeConfig(is_config=True, or_op=["gitlab_api_key", "github_api_key"])]
 
 
 class ReadPRDiffsOutputs(TypedDict):