rename ZIG_DEBUG_COLOR env variable to NO_COLOR

Author: dweiller

doc/docgen.zig

@@ -1305,7 +1305,7 @@ fn genHtml(
     defer root_node.end();
 
     var env_map = try process.getEnvMap(allocator);
-    try env_map.put("ZIG_DEBUG_COLOR", "1");
+    try env_map.put("NO_COLOR", "1");
 
     const host = try std.zig.system.NativeTargetInfo.detect(.{});
     const builtin_code = try getBuiltinCode(allocator, &env_map, zig_exe, opt_zig_lib_dir);

lib/build_runner.zig

@@ -282,7 +282,7 @@ pub fn main() !void {
     const ttyconf = get_tty_conf(color, stderr);
     switch (ttyconf) {
         .no_color => try builder.env_map.put("NO_COLOR", "1"),
-        .escape_codes => try builder.env_map.put("ZIG_DEBUG_COLOR", "1"),
+        .escape_codes => try builder.env_map.put("NO_COLOR", "1"),
         .windows_api => {},
     }
 

lib/std/io/tty.zig

@@ -7,11 +7,11 @@ const native_os = builtin.os.tag;
 
 /// Detect suitable TTY configuration options for the given file (commonly stdout/stderr).
 /// This includes feature checks for ANSI escape codes and the Windows console API, as well as
-/// respecting the `ZIG_DEBUG_COLOR` and `YES_COLOR` environment variables to override the default.
+/// respecting the `NO_COLOR` and `YES_COLOR` environment variables to override the default.
 pub fn detectConfig(file: File) Config {
     const force_color: ?bool = if (builtin.os.tag == .wasi)
         null // wasi does not support environment variables
-    else if (process.hasEnvVarConstant("ZIG_DEBUG_COLOR"))
+    else if (process.hasEnvVarConstant("NO_COLOR"))
         false
     else if (process.hasEnvVarConstant("YES_COLOR"))
         true

test/src/StackTrace.zig

@@ -81,7 +81,7 @@ fn addExpect(
     });
 
     const run = b.addRunArtifact(exe);
-    run.removeEnvironmentVariable("ZIG_DEBUG_COLOR");
+    run.removeEnvironmentVariable("NO_COLOR");
     run.setEnvironmentVariable("NO_COLOR", "1");
     run.expectExitCode(1);
     run.expectStdOutEqual("");