Ensure all rich tags are closed (#2585)

Fixes: #2571
ansible · Oct 10, 2022 · 66f961c · 66f961c
1 parent bda8d99
commit 66f961c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/ansiblelint/app.py b/src/ansiblelint/app.py
@@ -281,7 +281,7 @@ def report_summary(  # pylint: disable=too-many-branches,too-many-locals
 
         msg += f": {summary.failures} failure(s), {summary.warnings} warning(s)"
         if summary.fixed:
-            msg += f", and fixed [/]{summary.fixed} issue(s)"
+            msg += f", and fixed {summary.fixed} issue(s)"
         msg += f" on {files_count} files."
 
         console_stderr.print(msg)

diff --git a/src/ansiblelint/formatters/__init__.py b/src/ansiblelint/formatters/__init__.py
@@ -88,9 +88,7 @@ def format(self, match: MatchError) -> str:
         result = (
             f"[filename]{self._format_path(match.filename or '')}[/][dim]:{match.position}:[/] "
             f"[{match.level}][bold]{self.escape(match.tag)}[/bold]"
-            f": {match.message}"
-            if not options.quiet
-            else "[/]"
+            f"{ f': {match.message}' if not options.quiet else '' }[/]"
         )
         if match.level != "error":
             result += f" [dim][{match.level}]({match.level})[/][/]"