Remove enrich as a dependency

.config/ansible-lint.spec

@@ -34,7 +34,6 @@ BuildRequires:  git
 %endif
 # Named based on fedora 35:
 Requires:       ansible-core >= 2.12.0
-Requires:       python3-enrich
 Requires:       python3-packaging
 Requires:       python3-pyyaml
 Requires:       python3-rich

.pre-commit-config.yaml

@@ -142,7 +142,6 @@ repos:
           - ansible-compat>=2.2.0
           - ansible-core
           - black
-          - enrich
           - filelock
           - flaky
           - pytest
@@ -170,7 +169,6 @@ repos:
           - ansible-core
           - black
           - docutils
-          - enrich
           - filelock
           - flaky
           - jsonschema>=4.9.0

requirements.txt

@@ -23,7 +23,6 @@ coverage-enable-subprocess==1.0
 cryptography==38.0.1
 dill==0.3.5.1
 docutils==0.17.1
-enrich==1.2.7
 execnet==1.9.0
 filelock==3.8.0
 flake8==5.0.4

setup.cfg

@@ -70,7 +70,6 @@ install_requires =
   ansible-compat>=2.2.1  # GPLv3
   ansible-core>=2.12.0  # GPLv3
   black>=22.1.0  # MIT
-  enrich>=1.2.6
   filelock  # The Unlicense
   jsonschema>=4.9.0  # MIT, version needed for improved errors
   packaging

src/ansiblelint/__main__.py

@@ -31,11 +31,10 @@
 import subprocess
 import sys
 from contextlib import contextmanager
-from typing import TYPE_CHECKING, Any, Callable, Iterator
+from typing import TYPE_CHECKING, Any, Callable, Iterator, TextIO
 
 from ansible_compat.config import ansible_version
 from ansible_compat.prerun import get_cache_dir
-from enrich.console import should_do_markup
 from filelock import FileLock, Timeout
 
 from ansiblelint import cli
@@ -350,5 +349,50 @@ def path_inject() -> None:
         raise RuntimeError("Failed to find ansible executable in PATH")
 
 
+# Based on Ansible implementation
+def to_bool(value: Any) -> bool:
+    """Return a bool for the arg."""
+    if value is None or isinstance(value, bool):
+        return bool(value)
+    if isinstance(value, str):
+        value = value.lower()
+    if value in ("yes", "on", "1", "true", 1):
+        return True
+    return False
+
+
+def should_do_markup(stream: TextIO = sys.stdout) -> bool:
+    """Decide about use of ANSI colors."""
+    py_colors = None
+
+    # https://xkcd.com/927/
+    for env_var in ["PY_COLORS", "CLICOLOR", "FORCE_COLOR", "ANSIBLE_FORCE_COLOR"]:
+        value = os.environ.get(env_var, None)
+        if value is not None:
+            py_colors = to_bool(value)
+            break
+
+    # If deliverately disabled colors
+    if os.environ.get("NO_COLOR", None):
+        return False
+
+    # User configuration requested colors
+    if py_colors is not None:
+        return to_bool(py_colors)
+
+    term = os.environ.get("TERM", "")
+    if "xterm" in term:
+        return True
+
+    if term == "dumb":
+        return False
+
+    # Use tty detection logic as last resort because there are numerous
+    # factors that can make isatty return a misleading value, including:
+    # - stdin.isatty() is the only one returning true, even on a real terminal
+    # - stderr returting false if user user uses a error stream coloring solution
+    return stream.isatty()
+
+
 if __name__ == "__main__":
     _run_cli_entrypoint()