xiaxinhang/pr-agent
1
0
Fork 0
mirror of https://github.com/qodo-ai/pr-agent.git synced 2025-07-05 05:10:38 +08:00
Code Issues Packages Projects Releases Wiki Activity

remove title and description

Browse Source
This commit is contained in:
mrT23
2023-07-13 16:26:00 +03:00
parent d1b6b3bc95
commit 9fadde388b
4 changed files with 1 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@ -183,7 +183,6 @@ Here is a quick overview of the different sub-tools of PR Reviewer:
- PR Analysis - PR Analysis
- Summarize main theme - Summarize main theme
- PR description and title
- PR type classification - PR type classification
- Is the PR covered by relevant tests - Is the PR covered by relevant tests
- Is this a focused PR - Is this a focused PR
@ -199,7 +198,6 @@ This is how a typical output of the PR Reviewer looks like:
#### PR Analysis #### PR Analysis
- 🎯 **Main theme:** Adding language extension handler and token handler - 🎯 **Main theme:** Adding language extension handler and token handler
- 🔍 **Description and title:** Yes
- 📌 **Type of PR:** Enhancement - 📌 **Type of PR:** Enhancement
- 🧪 **Relevant tests added:** No - 🧪 **Relevant tests added:** No
- ✨ **Focused PR:** Yes, the PR is focused on adding two new handlers for language extension and token counting. - ✨ **Focused PR:** Yes, the PR is focused on adding two new handlers for language extension and token counting.

1
pr_agent/algo/utils.py
View File

@ -11,7 +11,6 @@ def convert_to_markdown(output_data: dict) -> str:
emojis = { emojis = {
"Main theme": "🎯", "Main theme": "🎯",
"Description and title": "🔍",
"Type of PR": "📌", "Type of PR": "📌",
"Relevant tests added": "🧪", "Relevant tests added": "🧪",
"Unrelated changes": "⚠️", "Unrelated changes": "⚠️",

7
pr_agent/settings/pr_reviewer_prompts.toml
View File

@ -13,10 +13,6 @@ You must use the following JSON schema to format your answer:
"type": "string", "type": "string",
"description": "a short explanation of the PR" "description": "a short explanation of the PR"
}, },
"Description and title": {
"type": "string",
"description": "yes\\no question: does this PR have a relevant description and title"
},
"Type of PR": { "Type of PR": {
"type": "string", "type": "string",
"enum": ["Bug fix", "Tests", "Bug fix with tests", "Refactoring", "Enhancement", "Documentation", "Other"] "enum": ["Bug fix", "Tests", "Bug fix with tests", "Refactoring", "Enhancement", "Documentation", "Other"]
@ -37,7 +33,7 @@ You must use the following JSON schema to format your answer:
"PR Feedback": { "PR Feedback": {
"General PR suggestions": { "General PR suggestions": {
"type": "string", "type": "string",
"description": "important suggestions for the contributors and maintainers of this PR, may include overall structure, primary purpose and best practices. consider using specific filenames, classes and functions names. explain yourself!" "description": "General important suggestions for the contributors and maintainers of this PR. May include suggestions for overall structure, primary purpose, best practices, etc. consider using specific filenames, classes and functions names. explain yourself!"
}, },
"Code suggestions": { "Code suggestions": {
"type": "array", "type": "array",
@ -75,7 +71,6 @@ Example output:
"PR Analysis": "PR Analysis":
{ {
"Main theme": "xxx", "Main theme": "xxx",
"Description and title": "Yes",
"Type of PR": "Bug fix", "Type of PR": "Bug fix",
{%- if require_tests %} {%- if require_tests %}
"Relevant tests added": "No", "Relevant tests added": "No",

3
tests/unit/test_convert_to_markdown.py
View File

@ -46,7 +46,6 @@ class TestConvertToMarkdown:
def test_simple_dictionary_input(self): def test_simple_dictionary_input(self):
input_data = { input_data = {
'Main theme': 'Test', 'Main theme': 'Test',
'Description and title': 'Test description',
'Type of PR': 'Test type', 'Type of PR': 'Test type',
'Relevant tests added': 'no', 'Relevant tests added': 'no',
'Unrelated changes': 'n/a', # won't be included in the output 'Unrelated changes': 'n/a', # won't be included in the output
@ -69,7 +68,6 @@ class TestConvertToMarkdown:
} }
expected_output = """\ expected_output = """\
- 🎯 **Main theme:** Test - 🎯 **Main theme:** Test
- 🔍 **Description and title:** Test description
- 📌 **Type of PR:** Test type - 📌 **Type of PR:** Test type
- 🧪 **Relevant tests added:** no - 🧪 **Relevant tests added:** no
- ✨ **Focused PR:** Yes - ✨ **Focused PR:** Yes
@ -108,7 +106,6 @@ class TestConvertToMarkdown:
def test_dictionary_input_containing_only_empty_dictionaries(self): def test_dictionary_input_containing_only_empty_dictionaries(self):
input_data = { input_data = {
'Main theme': {}, 'Main theme': {},
'Description and title': {},
'Type of PR': {}, 'Type of PR': {},
'Relevant tests added': {}, 'Relevant tests added': {},
'Unrelated changes': {}, 'Unrelated changes': {},