fix(cli): don't add colors for non-tty outputs (denoland#13031)

matthewleon · Dec 10, 2021 · 38f1630 · 38f1630
1 parent 542b71e
commit 38f1630
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/runtime/colors.rs b/runtime/colors.rs
@@ -9,11 +9,14 @@ use termcolor::{Ansi, ColorSpec, WriteColor};
 use termcolor::{BufferWriter, ColorChoice};
 
 lazy_static::lazy_static! {
+// checks if the output is piped to a tty
+  static ref IS_TTY: bool = atty::is(atty::Stream::Stdout);
   static ref NO_COLOR: bool = std::env::var_os("NO_COLOR").is_some();
 }
 
+// if the output is piped to a tty, use color
 pub fn use_color() -> bool {
-  !(*NO_COLOR)
+  !(*NO_COLOR) && *IS_TTY
 }
 
 #[cfg(windows)]