Add logging for empty PR files and update warning for empty diffs in pr_reviewer.py

This commit is contained in:
mrT23 2024-08-18 08:14:37 +03:00
parent dc1a8e8314
commit 5954c7cec2
No known key found for this signature in database
GPG key ID: D350490E39D5F5AD

View file

@ -1,5 +1,6 @@
import copy import copy
import datetime import datetime
import traceback
from collections import OrderedDict from collections import OrderedDict
from functools import partial from functools import partial
from typing import List, Tuple from typing import List, Tuple
@ -96,6 +97,10 @@ class PRReviewer:
async def run(self) -> None: async def run(self) -> None:
try: try:
if not self.git_provider.get_files():
get_logger().info(f"PR has no files: {self.pr_url}, skipping review")
return None
if self.incremental.is_incremental and not self._can_run_incremental_review(): if self.incremental.is_incremental and not self._can_run_incremental_review():
return None return None
@ -158,7 +163,7 @@ class PRReviewer:
get_logger().debug(f"PR diff", diff=self.patches_diff) get_logger().debug(f"PR diff", diff=self.patches_diff)
self.prediction = await self._get_prediction(model) self.prediction = await self._get_prediction(model)
else: else:
get_logger().error(f"Error getting PR diff") get_logger().warning(f"Empty diff for PR: {self.pr_url}")
self.prediction = None self.prediction = None
async def _get_prediction(self, model: str) -> str: async def _get_prediction(self, model: str) -> str: