search

sharkdp / hexyl

A command-line hex viewer
Apache License 2.0
8.92k stars 227 forks source link

Add support for the NO_COLOR environment variable #179

Closed mjf closed 6 months ago

mjf commented 1 year ago

Please, add support for the NO_COLOR environment variable, if possible.

sharkdp commented 1 year ago

Sure, sounds reasonable.

mainrs commented 1 year ago

This can be done by using another dependency for colored text instead of the current one. owo-colors supports the env variable natively and has zero overhead printing colored text.

I can work on this issue if that's ok.

nagy commented 8 months ago

Is it possible, that this has not really been fixed? When I run NO_COLOR=1 hexyl <<< hi I still see colored output. Am I doing something wrong? That is on hexyl version 0.13.1.

sharkdp commented 8 months ago

Strange. Doesn't work for me either. Re-opening for now.