From 95df26c97390d4296c443e79c406bbba7504fee1 Mon Sep 17 00:00:00 2001 From: Seohyun Lee Date: Tue, 20 May 2025 02:08:41 +0900 Subject: [PATCH 1/6] docs: Correct typos in ignore section configuration --- docs/docs/usage-guide/additional_configurations.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/docs/usage-guide/additional_configurations.md b/docs/docs/usage-guide/additional_configurations.md index eb46bbaa..9f9202f6 100644 --- a/docs/docs/usage-guide/additional_configurations.md +++ b/docs/docs/usage-guide/additional_configurations.md @@ -50,7 +50,7 @@ glob = ['*.py'] And to ignore Python files in all PRs using `regex` pattern, set in a configuration file: ``` -[regex] +[ignore] regex = ['.*\.py$'] ``` From 2a42d009afcfd60c33f3533314bc383dcb86b7a2 Mon Sep 17 00:00:00 2001 From: Seohyun Lee Date: Tue, 20 May 2025 02:27:14 +0900 Subject: [PATCH 2/6] docs: Fix broken Markdown code block in Installation documentation --- docs/docs/installation/github.md | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/docs/docs/installation/github.md b/docs/docs/installation/github.md index 018499ee..3eeace4f 100644 --- a/docs/docs/installation/github.md +++ b/docs/docs/installation/github.md @@ -193,9 +193,8 @@ For example: `GITHUB.WEBHOOK_SECRET` --> `GITHUB__WEBHOOK_SECRET` 3. Push image to ECR ```shell - - docker tag codiumai/pr-agent:serverless .dkr.ecr..amazonaws.com/codiumai/pr-agent:serverless - docker push .dkr.ecr..amazonaws.com/codiumai/pr-agent:serverless + docker tag codiumai/pr-agent:serverless .dkr.ecr..amazonaws.com/codiumai/pr-agent:serverless + docker push .dkr.ecr..amazonaws.com/codiumai/pr-agent:serverless ``` 4. Create a lambda function that uses the uploaded image. Set the lambda timeout to be at least 3m. From f5a069d6b4a41e77ceead860d3d151ab476dd648 Mon Sep 17 00:00:00 2001 From: Seohyun Lee Date: Tue, 20 May 2025 02:28:43 +0900 Subject: [PATCH 3/6] refactor: Align comment formatting with others --- pr_agent/tools/pr_reviewer.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pr_agent/tools/pr_reviewer.py b/pr_agent/tools/pr_reviewer.py index ff48819f..ac57d85d 100644 --- a/pr_agent/tools/pr_reviewer.py +++ b/pr_agent/tools/pr_reviewer.py @@ -316,7 +316,9 @@ class PRReviewer: get_logger().exception(f"Failed to remove previous review comment, error: {e}") def _can_run_incremental_review(self) -> bool: - """Checks if we can run incremental review according the various configurations and previous review""" + """ + Checks if we can run incremental review according the various configurations and previous review. + """ # checking if running is auto mode but there are no new commits if self.is_auto and not self.incremental.first_new_commit_sha: get_logger().info(f"Incremental review is enabled for {self.pr_url} but there are no new commits") From 94aa8e8638eae3b181ae0db0d991942cbbc2bdf8 Mon Sep 17 00:00:00 2001 From: Seohyun Lee Date: Tue, 20 May 2025 02:30:17 +0900 Subject: [PATCH 4/6] refactor: Correct case typo in log --- pr_agent/tools/pr_description.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pr_agent/tools/pr_description.py b/pr_agent/tools/pr_description.py index 6ab13ee1..df82db67 100644 --- a/pr_agent/tools/pr_description.py +++ b/pr_agent/tools/pr_description.py @@ -199,7 +199,7 @@ class PRDescription: async def _prepare_prediction(self, model: str) -> None: if get_settings().pr_description.use_description_markers and 'pr_agent:' not in self.user_description: - get_logger().info("Markers were enabled, but user description does not contain markers. skipping AI prediction") + get_logger().info("Markers were enabled, but user description does not contain markers. Skipping AI prediction") return None large_pr_handling = get_settings().pr_description.enable_large_pr_handling and "pr_description_only_files_prompts" in get_settings() @@ -707,7 +707,7 @@ class PRDescription: pr_body += """""" except Exception as e: - get_logger().error(f"Error processing pr files to markdown {self.pr_id}: {str(e)}") + get_logger().error(f"Error processing PR files to markdown {self.pr_id}: {str(e)}") pass return pr_body, pr_comments From 4679dce3afb49753d0c78f74b2373ccd25bf4787 Mon Sep 17 00:00:00 2001 From: Seohyun Lee Date: Tue, 20 May 2025 02:31:48 +0900 Subject: [PATCH 5/6] refactor: Correct case typo of PR in log --- pr_agent/git_providers/azuredevops_provider.py | 2 +- pr_agent/git_providers/github_provider.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pr_agent/git_providers/azuredevops_provider.py b/pr_agent/git_providers/azuredevops_provider.py index 35165bdd..d71a029f 100644 --- a/pr_agent/git_providers/azuredevops_provider.py +++ b/pr_agent/git_providers/azuredevops_provider.py @@ -618,7 +618,7 @@ class AzureDevopsProvider(GitProvider): return pr_id except Exception as e: if get_settings().config.verbosity_level >= 2: - get_logger().info(f"Failed to get pr id, error: {e}") + get_logger().info(f"Failed to get PR id, error: {e}") return "" def publish_file_comments(self, file_comments: list) -> bool: diff --git a/pr_agent/git_providers/github_provider.py b/pr_agent/git_providers/github_provider.py index 4f3a5ec7..fa52b7dc 100644 --- a/pr_agent/git_providers/github_provider.py +++ b/pr_agent/git_providers/github_provider.py @@ -96,7 +96,7 @@ class GithubProvider(GitProvider): parsed_url = urlparse(given_url) repo_path = (parsed_url.path.split('.git')[0])[1:] # //.git -> / if not repo_path: - get_logger().error(f"url is neither an issues url nor a pr url nor a valid git url: {given_url}. Returning empty result.") + get_logger().error(f"url is neither an issues url nor a PR url nor a valid git url: {given_url}. Returning empty result.") return "" return repo_path except Exception as e: From 25b807f71c20559a640c00129276abc3c74158d6 Mon Sep 17 00:00:00 2001 From: Seohyun Lee Date: Tue, 20 May 2025 03:11:21 +0900 Subject: [PATCH 6/6] docs: Add missing spacebar --- docs/docs/usage-guide/configuration_options.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/docs/usage-guide/configuration_options.md b/docs/docs/usage-guide/configuration_options.md index ab499dca..b9b3d5d9 100644 --- a/docs/docs/usage-guide/configuration_options.md +++ b/docs/docs/usage-guide/configuration_options.md @@ -5,7 +5,7 @@ See the [Tools Guide](https://qodo-merge-docs.qodo.ai/tools/) for a detailed des There are three main ways to set persistent configurations: -1. [Wiki](https://qodo-merge-docs.qodo.ai/usage-guide/configuration_options/#wiki-configuration-file)configuration page 💎 +1. [Wiki](https://qodo-merge-docs.qodo.ai/usage-guide/configuration_options/#wiki-configuration-file) configuration page 💎 2. [Local](https://qodo-merge-docs.qodo.ai/usage-guide/configuration_options/#local-configuration-file) configuration file 3. [Global](https://qodo-merge-docs.qodo.ai/usage-guide/configuration_options/#global-configuration-file) configuration file 💎