xiaxinhang/pr-agent
1
0
Fork 0
mirror of https://github.com/qodo-ai/pr-agent.git synced 2025-07-13 01:00:39 +08:00
Code Issues Packages Projects Releases Wiki Activity

backward compatibility

Browse Source
This commit is contained in:
Hussam.lawen
2024-04-17 09:19:26 +03:00
parent 6c5d434dcd
commit 2c397c28e5
2 changed files with 16 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
pr_agent/tools/pr_description.py
View File

@ -95,7 +95,10 @@ class PRDescription:
self.file_label_dict = self._prepare_file_labels() self.file_label_dict = self._prepare_file_labels()
pr_labels, pr_file_changes = [], [] pr_labels, pr_file_changes = [], []
if not get_settings().pr_description.disable_publish_labels: # backward compatibility support
publish_label = get_settings().pr_reviewer.get("publish_labels", True) \
and not get_settings().pr_description.disable_publish_labels
if publish_label:
pr_labels = self._prepare_labels() pr_labels = self._prepare_labels()
if get_settings().pr_description.use_description_markers: if get_settings().pr_description.use_description_markers:
@ -118,7 +121,7 @@ class PRDescription:
if get_settings().config.publish_output: if get_settings().config.publish_output:
# publish labels # publish labels
if not get_settings().pr_description.disable_publish_labels and self.git_provider.is_supported("get_labels"): if publish_label and self.git_provider.is_supported("get_labels"):
original_labels = self.git_provider.get_pr_labels(update=True) original_labels = self.git_provider.get_pr_labels(update=True)
get_logger().debug(f"original labels", artifact=original_labels) get_logger().debug(f"original labels", artifact=original_labels)
user_labels = get_user_labels(original_labels) user_labels = get_user_labels(original_labels)
@ -143,8 +146,10 @@ class PRDescription:
else: else:
self.git_provider.publish_description(pr_title, pr_body) self.git_provider.publish_description(pr_title, pr_body)
# publish final update message # publish final update message - with backward compatibility support
if (not get_settings().pr_description.disable_final_update_message): publish_update_message = get_settings().pr_reviewer.get("final_update_message", True)\
and not get_settings().pr_description.disable_final_update_message
if (publish_update_message):
latest_commit_url = self.git_provider.get_latest_commit_url() latest_commit_url = self.git_provider.get_latest_commit_url()
if latest_commit_url: if latest_commit_url:
pr_url = self.git_provider.get_pr_url() pr_url = self.git_provider.get_pr_url()
@ -301,7 +306,10 @@ class PRDescription:
# Remove the 'PR Title' key from the dictionary # Remove the 'PR Title' key from the dictionary
ai_title = self.data.pop('title', self.vars["title"]) ai_title = self.data.pop('title', self.vars["title"])
if not get_settings().pr_description.generate_ai_title: # get config - with backward compatibility
keep_user_title = get_settings().pr_reviewer.get("keep_original_user_title", True) \
and not get_settings().pr_description.generate_ai_title
if keep_user_title:
# Assign the original PR title to the 'title' variable # Assign the original PR title to the 'title' variable
title = self.vars["title"] title = self.vars["title"]
else: else:

4
pr_agent/tools/pr_reviewer.py
View File

@ -135,7 +135,9 @@ class PRReviewer:
if get_settings().config.publish_output: if get_settings().config.publish_output:
# publish the review # publish the review
if get_settings().pr_reviewer.persistent_comment and not self.incremental.is_incremental: if get_settings().pr_reviewer.persistent_comment and not self.incremental.is_incremental:
final_update_message = not get_settings().pr_reviewer.disable_final_update_message # get config with backward compatibility
final_update_message = get_settings().pr_reviewer.get("final_update_message", True)\
and not get_settings().pr_reviewer.disable_final_update_message
self.git_provider.publish_persistent_comment(pr_review, self.git_provider.publish_persistent_comment(pr_review,
initial_header="## PR Review", initial_header="## PR Review",
update_header=True, update_header=True,