You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We should probably just use noColor as it is essentially an opt-out preference; color could imply the user wants color which could not be true (rather than NO_COLOR stating the user explicitly does not want color). Also, this would then share the name of the environment variable "standard".
Regarding NO_COLOR:
On the opposite side of NO_COLOR is FORCE_COLOR, definitely less known, but respected by some big npm packages, e.g. chalk and supports-color.
I don't think there's described behavior about connecting FORCE_COLOR and NO_COLOR, so this is kind of ambiguous.
I'm definitely against names hasColor or colorful (both imply that terminal can display color).
I don't see having both noColor and forceColor as an issue though.
a boolean flag for if color is supported ignores that some terminals support different amounts of color detail. for example, the npm package supports-color has four levels of color support to determine what set of colors are supported. though, these checks are very heuristics based and standardizing it would probably be a micro-nightmare.
Terminal metadata idea from this #1 comment.
CLI.terminal.noColor
) or bare in the namespace (egCLI.noColor
)?CLI.terminal.noColor()
) or getters (egCLI.terminal.noColor
)?interactive
,isInteractive
,interactivity
,nonInteractive
,notInteractive
,tty
,isTty
,isTTY
,terminal
,isTerminal
,inTerminal
isatty(...)
function, but now is moving toisTerminal
.noColor
,color
,colorful
,hasColor
,shouldColor
noColor
as it is essentially an opt-out preference;color
could imply the user wants color which could not be true (rather thanNO_COLOR
stating the user explicitly does not want color). Also, this would then share the name of the environment variable "standard".Deno.noColor
) afaik, other runtimes you have to handle it yourself (eg!!process.env.NO_COLOR
).The text was updated successfully, but these errors were encountered: