Estimated effort to review

This commit is contained in:
mrT23
2023-09-17 16:31:58 +03:00
parent caaee4e43d
commit ea91a38541
4 changed files with 7 additions and 7 deletions

View File

@ -42,7 +42,7 @@ def convert_to_markdown(output_data: dict, gfm_supported: bool=True) -> str:
"General suggestions": "💡", "General suggestions": "💡",
"Insights from user's answers": "📝", "Insights from user's answers": "📝",
"Code feedback": "🤖", "Code feedback": "🤖",
"Estimated time to review": "⏱️", "Estimated effort to review": "⏱️",
} }
for key, value in output_data.items(): for key, value in output_data.items():

View File

@ -18,7 +18,7 @@ require_focused_review=false
require_score_review=false require_score_review=false
require_tests_review=true require_tests_review=true
require_security_review=true require_security_review=true
require_estimate_time_to_review=false require_estimate_effort_to_review=true
num_code_suggestions=4 num_code_suggestions=4
inline_code_comments = false inline_code_comments = false
ask_and_reflect=false ask_and_reflect=false

View File

@ -85,12 +85,12 @@ PR Analysis:
code diff changes are too scattered, then the PR is not focused. Explain code diff changes are too scattered, then the PR is not focused. Explain
your answer shortly. your answer shortly.
{%- endif %} {%- endif %}
{%- if require_estimate_time_to_review %} {%- if require_estimate_effort_to_review %}
Estimated time to review: Estimated effort to review [1-5]:
type: string type: string
description: >- description: >-
Estimate, in minutes, how much time it would take for an experienced developer to review this PR, and provide meaningful feedback. Estimate, on a scale of 1-5 (inclusive), the time and effort required to review this PR by an experienced and knowledgeable developer. 1 means short and easy review , 5 means long and hard review.
Take into account the size, complexity, quality and possible needed changes of the PR code diff. Take into account the size, complexity, quality, and the needed changes of the PR code diff.
Explain your answer shortly (1-2 sentences). Explain your answer shortly (1-2 sentences).
{%- endif %} {%- endif %}
PR Feedback: PR Feedback:

View File

@ -59,7 +59,7 @@ class PRReviewer:
"require_tests": get_settings().pr_reviewer.require_tests_review, "require_tests": get_settings().pr_reviewer.require_tests_review,
"require_security": get_settings().pr_reviewer.require_security_review, "require_security": get_settings().pr_reviewer.require_security_review,
"require_focused": get_settings().pr_reviewer.require_focused_review, "require_focused": get_settings().pr_reviewer.require_focused_review,
"require_estimate_time_to_review": get_settings().pr_reviewer.require_estimate_time_to_review, "require_estimate_effort_to_review": get_settings().pr_reviewer.require_estimate_effort_to_review,
'num_code_suggestions': get_settings().pr_reviewer.num_code_suggestions, 'num_code_suggestions': get_settings().pr_reviewer.num_code_suggestions,
'question_str': question_str, 'question_str': question_str,
'answer_str': answer_str, 'answer_str': answer_str,