From 1b3fb49f9c49367cd82fec1d8d73721382280231 Mon Sep 17 00:00:00 2001 From: mrT23 Date: Sun, 8 Oct 2023 16:50:25 +0300 Subject: [PATCH] publish each suggestion seperatly only on gitlab --- pr_agent/git_providers/gitlab_provider.py | 4 ++++ pr_agent/tools/pr_code_suggestions.py | 4 ++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/pr_agent/git_providers/gitlab_provider.py b/pr_agent/git_providers/gitlab_provider.py index 2e6f2140..de3762e6 100644 --- a/pr_agent/git_providers/gitlab_provider.py +++ b/pr_agent/git_providers/gitlab_provider.py @@ -196,6 +196,10 @@ class GitLabProvider(GitProvider): return self.last_diff # fallback to last_diff if no relevant diff is found def publish_code_suggestions(self, code_suggestions: list) -> bool: + if len(code_suggestions) > 1: + logging.debug(f"in gitlab we dont currently support publishing multiple code suggestions simultaneously") + return False + for suggestion in code_suggestions: try: body = suggestion['body'] diff --git a/pr_agent/tools/pr_code_suggestions.py b/pr_agent/tools/pr_code_suggestions.py index e1d5206c..d49459b0 100644 --- a/pr_agent/tools/pr_code_suggestions.py +++ b/pr_agent/tools/pr_code_suggestions.py @@ -138,8 +138,8 @@ class PRCodeSuggestions: if get_settings().config.verbosity_level >= 2: logging.info(f"Could not parse suggestion: {d}") - # is_successful = self.git_provider.publish_code_suggestions(code_suggestions) - if True: + is_successful = self.git_provider.publish_code_suggestions(code_suggestions) + if not is_successful: # logging.info("Failed to publish code suggestions, trying to publish each suggestion separately") for code_suggestion in code_suggestions: self.git_provider.publish_code_suggestions([code_suggestion])