Merge pull request #1043 from KennyDizi/main

Refactor markdown generation logic to eliminate redundancy
This commit is contained in:
Tal
2024-07-17 10:04:24 +03:00
committed by GitHub

View File

@ -156,17 +156,10 @@ def convert_to_markdown_v2(output_data: dict,
markdown_text += f"{emoji}&nbsp;<strong>PR contains tests</strong>" markdown_text += f"{emoji}&nbsp;<strong>PR contains tests</strong>"
markdown_text += f"</td></tr>\n" markdown_text += f"</td></tr>\n"
else: else:
if gfm_supported: if is_value_no(value):
markdown_text += f"<tr><td>" markdown_text += f'### {emoji} No relevant tests\n\n'
if is_value_no(value):
markdown_text += f"{emoji}&nbsp;<strong>No relevant tests</strong>"
else:
markdown_text += f"{emoji}&nbsp;<strong>PR contains tests</strong>"
else: else:
if is_value_no(value): markdown_text += f"### PR contains tests\n\n"
markdown_text += f'### {emoji} No relevant tests\n\n'
else:
markdown_text += f"### PR contains tests\n\n"
elif 'security concerns' in key_nice.lower(): elif 'security concerns' in key_nice.lower():
if gfm_supported: if gfm_supported:
markdown_text += f"<tr><td>" markdown_text += f"<tr><td>"