Don't use colors when stdout or stderr are being redirected

SpyrosRoum · Oct 6, 2021 · e7dd6f2 · e7dd6f2
1 parent f713954
commit e7dd6f2
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 2 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -13,6 +13,7 @@ description = "A command-line utility for easily compressing and decompressing f
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [dependencies]
+atty        = "0.2.14"
 lazy_static = "1.4.0"
 walkdir     = "2.3.2"
 strsim      = "0.10.0"

diff --git a/src/lib.rs b/src/lib.rs
@@ -29,7 +29,7 @@ lazy_static! {
     static ref NO_COLOR_IS_SET: bool = {
         use std::env;
 
-        env::var("NO_COLOR").is_ok()
+        env::var("NO_COLOR").is_ok() || atty::isnt(atty::Stream::Stdout) || atty::isnt(atty::Stream::Stderr)
     };
 }
 

diff --git a/src/utils.rs b/src/utils.rs
@@ -91,7 +91,7 @@ pub mod colors {
 #[allow(dead_code, non_upper_case_globals)]
 #[cfg(not(target_family = "unix"))]
 pub mod colors {
-    pub fn empty() -> &'static str {
+    pub const fn empty() -> &'static str {
         ""
     }
     pub const reset: fn() -> &'static str = empty;