Adjust markdown content for scans reporting (#2482)

Adjust markdown content for scans reporting TODO: link the discussion Reviewed-by: Kamil Dudka <[email protected]> Reviewed-by: Siteshwar Vashisht Reviewed-by: Maja Massarini
packit · Aug 5, 2024 · d8f9317 · d8f9317
2 parents 54858a7 + ab99505
commit d8f9317
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/packit_service/worker/handlers/copr.py b/packit_service/worker/handlers/copr.py
@@ -584,7 +584,9 @@ def handle_scan(self):
                 check_names=["osh-diff-scan:fedora-rawhide-x86_64"],
                 markdown_content=(
                     "This is an experimental feature. Once the scan finishes, you can see the "
-                    "newly introduced defects in the `added.html` in `Logs`. "
+                    "new findings in the `added.html` in `Logs`. \n\n"
+                    ":warning: You can see the list of known issues and also provide your feedback"
+                    " [here](https://github.com/packit/packit/discussions/2371). \n\n"
                     "You can disable the scanning in your configuration by "
                     "setting `osh_diff_scan_after_copr_build` to `false`. For more information, "
                     f"see [docs]({DOCS_URL}/configuration#osh_diff_scan_after_copr_build)."

diff --git a/packit_service/worker/reporting/reporters/github.py b/packit_service/worker/reporting/reporters/github.py
@@ -107,7 +107,7 @@ def set_status(
         summary = (
             self._create_table(url, links_to_external_services)
             + markdown_content
-            + "\n"
+            + "\n\n"
             + f"---\n*{News.get_sentence()}*"
         )
 

diff --git a/tests/unit/test_reporting.py b/tests/unit/test_reporting.py
@@ -244,7 +244,7 @@ def test_set_status_github_check(
         status=check_status,
         conclusion=check_conclusion,
         output=create_github_check_run_output(
-            title, summary + "\n---\n*Interesting news.*"
+            title, summary + "\n\n---\n*Interesting news.*"
         ),
     ).once()
 
@@ -460,7 +460,7 @@ def test_status_instead_check(
         status=check_status,
         conclusion=check_conclusion,
         output=create_github_check_run_output(
-            title, summary + "\n---\n*Interesting news.*"
+            title, summary + "\n\n---\n*Interesting news.*"
         ),
     ).and_raise(exception_type).once()