make nonempty testing easier since env variables always string|undefined

alexeyraspopov · Oct 16, 2024 · 65079be · 65079be
1 parent 3cfbed8
commit 65079be
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/picocolors.js b/picocolors.js
@@ -1,14 +1,13 @@
 let p = process || {},
 	argv = p.argv || [],
 	env = p.env || {}
-let ne = v => v != null && String(v).length > 0
 let isColorSupported =
-	!(ne(env.NO_COLOR) || argv.includes("--no-color")) &&
-	(ne(env.FORCE_COLOR) ||
+	!(!!env.NO_COLOR || argv.includes("--no-color")) &&
+	(!!env.FORCE_COLOR ||
 		argv.includes("--color") ||
 		p.platform === "win32" ||
 		((p.stdout || {}).isTTY && env.TERM !== "dumb") ||
-		ne(env.CI))
+		!!env.CI)
 
 let formatter =
 	(open, close, replace = open) =>