Skip to content

repl: fix `NO_COLORS` env var is ignored

Rodrigo Muino Tomonari requested to merge github/fork/MoLow/fix-no-colors into main

refs: https://github.com/nodejs/node/pull/48034#issuecomment-1904710409

the logic in options.useColors = shouldColorize(options.output) || process.env.NODE_DISABLE_COLORS === undefined; indroduced this bug since shouldColorize already checks NODE_DISABLE_COLORS among other env vars that disable colors: NO_COLOR and TERM === 'dumb'

Merge request reports