add-security-linter-results-to-workflow-summary

[Jannis-Mittenzwei]

closes #248

[Nicoretti]

Suggested change

	def issues_to_markdown(issues: Iterable[Issue]) -> str:
	markdown_str = ""
	markdown_str += "# Security\n\n"
	markdown_str += "|File|Cve|Cwe|Details|\n"
	markdown_str += "|---|:-:|:-:|---|\n"
	for issue in issues:
	row = "|" + issue.coordinates + "|"
	row += issue.cve + "|"
	row += issue.cwe + "|"
	for element in issue.references:
	row += element + " ,<br>"
	markdown_str += row[:-5] + "|\n"
	return markdown_str
	from inspect import cleandoc
	def issues_to_markdown(issues: Iterable[Issue]) -> str:
	template = cleandoc("""
	{header}
	{rows}
	""")
	def _header():
	header = ""
	markdown_str += "# Security\n\n"
	header += "|File|Cve|Cwe|Details|\n"
	header += "|---|:-:|:-:|---|\n"
	return header
	def _row(issue):
	row = "|" + issue.coordinates + "|"
	row += issue.cve + "|"
	row += issue.cwe + "|"
	for element in issue.references:
	row += element + " ,<br>"
	row += row[:-5] + "|"
	return row
	return template.format(
	header=_header()
	rows = "\n".jon(_entry(i) for i in issues)
	)

[Nicoretti]

Additionally if you make use of "".join() you can safe yourself the head egg with removing the bits from the last line here:

        for element in issue.references:
            row += element + " ,<br>"
        row += row[:-5] + "|"

Similar to what is done in the new return statement for rows

[Nicoretti]

f'{issue["line_number"]}' has the same effect as str(issue["line_number"])

[Nicoretti]

Suggested change

	str(PROJECT_CONFIG.root) + "/", ""
	) + ":" + str(issue["line_number"]) + ":" + str(issue["col_offset"]) + ":",
	f'{PROJECT_CONFIG.root}/, ""') + f':{issue["line_number"]}:{issue["col_offset"]}:',