From 8adfca5b3c16130eef12d405574d5e8f246240bf Mon Sep 17 00:00:00 2001 From: mrT23 Date: Wed, 14 Aug 2024 07:54:11 +0300 Subject: [PATCH] Add error handling for short diffs and improve logging in Bitbucket provider --- pr_agent/git_providers/bitbucket_provider.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/pr_agent/git_providers/bitbucket_provider.py b/pr_agent/git_providers/bitbucket_provider.py index 530cd670..1c990dba 100644 --- a/pr_agent/git_providers/bitbucket_provider.py +++ b/pr_agent/git_providers/bitbucket_provider.py @@ -192,9 +192,12 @@ class BitbucketProvider(GitProvider): else: if diffs[i].data.get('lines_added', 0) == 0 and diffs[i].data.get('lines_removed', 0) == 0: diff_split[i] = "" + elif len(diff_split_lines) <= 3: + diff_split[i] = "" + get_logger().info(f"Disregarding empty diff for file {_gef_filename(diffs[i])}") else: - get_logger().error(f"Error - failed to remove the bitbucket header from diff {i}") - break + get_logger().error(f"Error - failed to get diff for file {_gef_filename(diffs[i])}") + diff_split[i] = "" invalid_files_names = [] diff_files = []