Skip to content

Fix policy bot comment layout

Martin Čavoj requested to merge mc/fix-bot-comment-layout into master

What does this MR do and why?

The layout of policy bot comment got broken in Update default bot comment text when approvals ... (!143717 - merged) due to additional whitespaces in markdown.

The markdown that was generated:

:warning: **Policy violation(s) detected**

        Security and compliance scanners enforced by your organization have completed and identified that approvals
        are required due to one or more policy violations.
        Review the policy's rules in the MR widget and assign reviewers to proceed.

        Several factors can lead to a violation or required approval in your merge request:

        - If merge request approval policies enforced on your project include a scanner in the conditions, the scanner must be properly configured to run in both the source and target branch, the required jobs must complete, and a job artifact containing the scan results must be produced (even if empty).
        - If any violation of a merge request approval policy's rules are found, approval is required.
- Approvals are assumed required until all pipelines associated with the merge base commit in the target branch, and all pipelines associated with the latest commit in the source branch, are complete and it's confirmed that no policy violations have occurred.

This MR fixes the whitespaces.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Before After
CleanShot_2024-02-20_at_16.23.21_2x CleanShot_2024-02-20_at_16.23.08_2x
Edited by Martin Čavoj

Merge request reports