We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
uvu --no-color
The cli arguments defines -c, --color Print colorized output (default true), allowing the option --no-color to disable output colorization.
-c, --color Print colorized output (default true)
--no-color
But using it has no effect.
This is due to two things:
The option set process.env.FORCE_COLOR = '1' when true (the default) but is never set to something else. See: https://github.com/lukeed/uvu/blob/master/bin.js#L22 Since the default is true I propose
process.env.FORCE_COLOR = '1'
true
if (opts.color) process.env.FORCE_COLOR = '1'; else process.env.FORCE_COLOR = '0';
process.env.FORCE_COLOR
colorization is done using kleur in https://github.com/lukeed/uvu/blob/master/src/diff.js but it does not check for process.env.FORCE_COLOR
kleur
kleur has a enabled option, which could be usefull:
enabled
if(process.env.FORCE_COLOR === '0') kleur.enabled = false
I've open #226 to fix this
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
The cli arguments defines
-c, --color Print colorized output (default true)
, allowing the option--no-color
to disable output colorization.But using it has no effect.
This is due to two things:
option is never checked for false
The option set
process.env.FORCE_COLOR = '1'
when true (the default) but is never set to something else.See: https://github.com/lukeed/uvu/blob/master/bin.js#L22
Since the default is
true
I proposeprocess.env.FORCE_COLOR
is never checked.colorization is done using
kleur
in https://github.com/lukeed/uvu/blob/master/src/diff.jsbut it does not check for
process.env.FORCE_COLOR
kleur has a
enabled
option, which could be usefull:I've open #226 to fix this
The text was updated successfully, but these errors were encountered: