Merge pull request #69 from dmerejkowsky/fix-66

Fix regression on non-Windows platforms

Author: dmerejkowsky

cli_ui/__init__.py

@@ -38,7 +38,11 @@
 _ENABLE_XTERM_TITLE = None
 
 
-colorama.init()
+if os.name == "nt":
+    # On Windows using `isatty()` does *not* work reliably,
+    # so we always setup colorama.
+    # See colors_enabled() for details
+    colorama.init()
 
 
 # Tokens can be strings, or Color, UnicodeSequence or Symbol instances,
@@ -655,8 +659,10 @@ def main_demo() -> None:
 
 def main() -> None:
     parser = argparse.ArgumentParser()
+    parser.add_argument("--color", choices=["always", "never", "auto"])
     parser.add_argument("action", choices=["test_colors", "demo"])
     args = parser.parse_args()
+    setup(color=args.color)
     if args.action == "demo":
         main_demo()
     elif args.action == "test_colors":

docs/changelog.rst

@@ -5,6 +5,7 @@ v0.14.1
 +++++++
 
 * Development branch is now called ``main``.
+* Fix #66 - only call ``colorama.init()`` on Windows.
 
 v0.14.0 (2021-04-20)
 ++++++++++++++++++++