Skip to content

Commit

Permalink
Merge pull request #1376 from pdecat/enhancement/generalize_publish_o…
Browse files Browse the repository at this point in the history
…utput_progress

Add publish_output_progress config support to AzureDevOps, BitBucket and Gitlab providers
  • Loading branch information
mrT23 authored Dec 2, 2024
2 parents b440301 + a825aec commit eabc296
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pr_agent/git_providers/azuredevops_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -378,6 +378,9 @@ def get_diff_files(self) -> list[FilePatchInfo]:
return []

def publish_comment(self, pr_comment: str, is_temporary: bool = False, thread_context=None):
if is_temporary and not get_settings().config.publish_output_progress:
get_logger().debug(f"Skipping publish_comment for temporary comment: {pr_comment}")
return None
comment = Comment(content=pr_comment)
thread = CommentThread(comments=[comment], thread_context=thread_context, status=5)
thread_response = self.azure_devops_client.create_thread(
Expand Down
3 changes: 3 additions & 0 deletions pr_agent/git_providers/bitbucket_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -325,6 +325,9 @@ def publish_persistent_comment(self, pr_comment: str,
self.publish_comment(pr_comment)

def publish_comment(self, pr_comment: str, is_temporary: bool = False):
if is_temporary and not get_settings().config.publish_output_progress:
get_logger().debug(f"Skipping publish_comment for temporary comment: {pr_comment}")
return None
pr_comment = self.limit_output_characters(pr_comment, self.max_comment_length)
comment = self.pr.comment(pr_comment)
if is_temporary:
Expand Down
3 changes: 3 additions & 0 deletions pr_agent/git_providers/gitlab_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,9 @@ def publish_persistent_comment(self, pr_comment: str,
self.publish_persistent_comment_full(pr_comment, initial_header, update_header, name, final_update_message)

def publish_comment(self, mr_comment: str, is_temporary: bool = False):
if is_temporary and not get_settings().config.publish_output_progress:
get_logger().debug(f"Skipping publish_comment for temporary comment: {mr_comment}")
return None
mr_comment = self.limit_output_characters(mr_comment, self.max_comment_chars)
comment = self.mr.notes.create({'body': mr_comment})
if is_temporary:
Expand Down

0 comments on commit eabc296

Please sign in to comment.