Merge commit '7abbe08ff15d31c7e20de6d88638bfe27430c2f4' into hl/incremental_review

This commit is contained in:
Hussam.lawen
2023-07-19 14:24:44 +03:00
3 changed files with 7 additions and 5 deletions

View File

@ -34,7 +34,8 @@ def convert_to_markdown(output_data: dict) -> str:
elif isinstance(value, list):
if key.lower() == 'code suggestions':
markdown_text += "\n" # just looks nicer with additional line breaks
emoji = emojis.get(key, "") # Use a dash if no emoji is found for the key
# emoji = emojis.get(key, "‣") # Use a dash if no emoji is found for the key
emoji = ""
markdown_text += f"- {emoji} **{key}:**\n\n"
for item in value:
if isinstance(item, dict) and key.lower() == 'code suggestions':
@ -42,7 +43,8 @@ def convert_to_markdown(output_data: dict) -> str:
elif item:
markdown_text += f" - {item}\n"
elif value != 'n/a':
emoji = emojis.get(key, "") # Use a dash if no emoji is found for the key
# emoji = emojis.get(key, "‣") # Use a dash if no emoji is found for the key
emoji = ""
markdown_text += f"- {emoji} **{key}:** {value}\n"
return markdown_text

View File

@ -8,7 +8,7 @@ verbosity_level=0 # 0,1,2
require_focused_review=true
require_tests_review=true
require_security_review=true
num_code_suggestions=3
num_code_suggestions=0
inline_code_comments = true
ask_and_reflect=false

View File

@ -103,7 +103,7 @@ Example output:
...
]
{%- endif %}
{%- if require_security %},
{%- if require_security %}
"Security concerns": "No, because ..."
{%- endif %}
}