Merge ~ec0/juju-lint:fix-logging-color-when-piped into juju-lint:master

[James Hebden]

Disable coloured logging when stdout is redirected

This merge ensures coloured output is disabled when the standard output of the program is redirected from the console (either piped to another program, or redirected to a file) in the user's shell, rather than using the commandline argument to create a log file.

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 9c9a145973e94d89f50108dd2f547b74bb1a5bb4

diff --git a/jujulint/logging.py b/jujulint/logging.py
index 00dd200..47eaa88 100644
--- a/jujulint/logging.py
+++ b/jujulint/logging.py
@@ -32,7 +32,7 @@ class Logger:
         if not len(self.logger.handlers):
             format_string = "%(log_color)s%(asctime)s [%(levelname)s] %(message)s"
             date_format = "%Y-%m-%d %H:%M:%S"
-            colour_formatter = colorlog.ColoredFormatter(
+            colour_formatter = colorlog.TTYColoredFormatter(
                 format_string,
                 datefmt=date_format,
                 log_colors={
@@ -42,6 +42,7 @@ class Logger:
                     "ERROR": "red",
                     "CRITICAL": "red,bg_white",
                 },
+                stream=sys.stdout,
             )
             console = colorlog.StreamHandler()
             console.setFormatter(colour_formatter)