From 677a54c5a07a836567707c30f3e5702318f19752 Mon Sep 17 00:00:00 2001
From: ofir-frd <85901822+ofir-frd@users.noreply.github.com>
Date: Fri, 14 Mar 2025 12:02:26 +0200
Subject: [PATCH 1/2] Update review.md
---
docs/docs/tools/review.md | 110 +++++++++++++++++++-------------------
1 file changed, 55 insertions(+), 55 deletions(-)
diff --git a/docs/docs/tools/review.md b/docs/docs/tools/review.md
index e3c3c3c6..37dfcc8e 100644
--- a/docs/docs/tools/review.md
+++ b/docs/docs/tools/review.md
@@ -51,68 +51,68 @@ extra_instructions = "..."
!!! example "General options"
-
-
- require_score_review |
- If set to true, the tool will add a section that scores the PR. Default is false. |
-
-
- require_tests_review |
- If set to true, the tool will add a section that checks if the PR contains tests. Default is true. |
-
-
- require_estimate_effort_to_review |
- If set to true, the tool will add a section that estimates the effort needed to review the PR. Default is true. |
-
-
- require_can_be_split_review |
- If set to true, the tool will add a section that checks if the PR contains several themes, and can be split into smaller PRs. Default is false. |
-
-
- require_security_review |
- If set to true, the tool will add a section that checks if the PR contains a possible security or vulnerability issue. Default is true. |
-
-
- require_ticket_analysis_review |
- If set to true, and the PR contains a GitHub or Jira ticket link, the tool will add a section that checks if the PR in fact fulfilled the ticket requirements. Default is true. |
-
-
+
+
+ require_score_review |
+ If set to true, the tool will add a section that scores the PR. Default is false. |
+
+
+ require_tests_review |
+ If set to true, the tool will add a section that checks if the PR contains tests. Default is true. |
+
+
+ require_estimate_effort_to_review |
+ If set to true, the tool will add a section that estimates the effort needed to review the PR. Default is true. |
+
+
+ require_can_be_split_review |
+ If set to true, the tool will add a section that checks if the PR contains several themes, and can be split into smaller PRs. Default is false. |
+
+
+ require_security_review |
+ If set to true, the tool will add a section that checks if the PR contains a possible security or vulnerability issue. Default is true. |
+
+
+ require_ticket_analysis_review |
+ If set to true, and the PR contains a GitHub or Jira ticket link, the tool will add a section that checks if the PR in fact fulfilled the ticket requirements. Default is true. |
+
+
!!! example "Adding PR labels"
-You can enable\disable the `review` tool to add specific labels to the PR:
+ You can enable\disable the `review` tool to add specific labels to the PR:
-
-
- publish_labels |
- If set to true, the tool will publish labels to the PR. Default is false. |
-
-
- publish_description_as_comment |
- If set to true, the tool will publish the description as a comment to the PR. If false, it will overwrite the original description. Default is false. |
-
-
- publish_description_as_comment_persistent |
- If set to true and `publish_description_as_comment` is true, the tool will publish the description as a persistent comment to the PR. Default is true. |
-
-
- add_original_user_description |
- If set to true, the tool will add the original user description to the generated description. Default is true. |
-
-
- generate_ai_title |
- If set to true, the tool will also generate an AI title for the PR. Default is false. |
-
-
- extra_instructions |
- Optional extra instructions to the tool. For example: "focus on the changes in the file X. Ignore change in ..." |
-
-
- enable_pr_type |
- If set to false, it will not show the `PR type` as a text value in the description content. Default is true. |
-
-
- final_update_message |
- If set to true, it will add a comment message [`PR Description updated to latest commit...`](https://github.com/Codium-ai/pr-agent/pull/499#issuecomment-1837412176) after finishing calling `/describe`. Default is false. |
-
-
- enable_semantic_files_types |
- If set to true, "Changes walkthrough" section will be generated. Default is true. |
-
-
- collapsible_file_list |
- If set to true, the file list in the "Changes walkthrough" section will be collapsible. If set to "adaptive", the file list will be collapsible only if there are more than 8 files. Default is "adaptive". |
-
-
- enable_large_pr_handling |
- Pro feature. If set to true, in case of a large PR the tool will make several calls to the AI and combine them to be able to cover more files. Default is true. |
-
-
- enable_help_text |
- If set to true, the tool will display a help text in the comment. Default is false. |
-
-
+
+
+ publish_labels |
+ If set to true, the tool will publish labels to the PR. Default is false. |
+
+
+ publish_description_as_comment |
+ If set to true, the tool will publish the description as a comment to the PR. If false, it will overwrite the original description. Default is false. |
+
+
+ publish_description_as_comment_persistent |
+ If set to true and `publish_description_as_comment` is true, the tool will publish the description as a persistent comment to the PR. Default is true. |
+
+
+ add_original_user_description |
+ If set to true, the tool will add the original user description to the generated description. Default is true. |
+
+
+ generate_ai_title |
+ If set to true, the tool will also generate an AI title for the PR. Default is false. |
+
+
+ extra_instructions |
+ Optional extra instructions to the tool. For example: "focus on the changes in the file X. Ignore change in ..." |
+
+
+ enable_pr_type |
+ If set to false, it will not show the `PR type` as a text value in the description content. Default is true. |
+
+
+ final_update_message |
+ If set to true, it will add a comment message [`PR Description updated to latest commit...`](https://github.com/Codium-ai/pr-agent/pull/499#issuecomment-1837412176) after finishing calling `/describe`. Default is false. |
+
+
+ enable_semantic_files_types |
+ If set to true, "Changes walkthrough" section will be generated. Default is true. |
+
+
+ collapsible_file_list |
+ If set to true, the file list in the "Changes walkthrough" section will be collapsible. If set to "adaptive", the file list will be collapsible only if there are more than 8 files. Default is "adaptive". |
+
+
+ enable_large_pr_handling |
+ Pro feature. If set to true, in case of a large PR the tool will make several calls to the AI and combine them to be able to cover more files. Default is true. |
+
+
+ enable_help_text |
+ If set to true, the tool will display a help text in the comment. Default is false. |
+
+
## Inline file summary 💎