Improve CI variable handling

news/5128.bugfix.rst

@@ -0,0 +1 @@
+If environment variable ``CI`` or ``TF_BUILD`` is set but does not evaluate to ``False`` it is now treated as ``True``.

pipenv/environments.py

@@ -81,7 +81,15 @@ def normalize_pipfile_path(p):
 os.environ.pop("__PYVENV_LAUNCHER__", None)
 # Internal, to tell whether the command line session is interactive.
 SESSION_IS_INTERACTIVE = _isatty(sys.stdout)
-PIPENV_IS_CI = env_to_bool(os.environ.get("CI") or os.environ.get("TF_BUILD") or False)
+
+# TF_BUILD indicates to Azure pipelines it is a build step
+PIPENV_IS_CI = os.environ.get("CI") or os.environ.get("TF_BUILD")
+try:
+    PIPENV_IS_CI = env_to_bool(PIPENV_IS_CI)
+except ValueError:
+    # CI variable detected and it did not evaluate to a false value
+    PIPENV_IS_CI = True
+
 NO_COLOR = False
 if os.getenv("NO_COLOR") or os.getenv("PIPENV_COLORBLIND"):
     NO_COLOR = True