Support `NO_COLOR` - Githubissues

dbuenzli / down

An OCaml toplevel (REPL) upgrade

http://erratique.ch/software/down

ISC License

84 stars 3 forks source link

Support `NO_COLOR` #32

Closed henrytill closed 10 months ago

henrytill commented 10 months ago

Hi, thanks for the great tool.

Would you be open to adding support for the NO_COLOR environment variable?

I sketched out a very basic approach here.

dbuenzli commented 10 months ago

If anything I'm rather sympathetic to add support for OCAML_COLOR and/or TERM=dumb. I think NO_COLOR is a bad standard, someone explained it better than me there.

henrytill commented 10 months ago

Okay, understood.

dbuenzli commented 10 months ago

and/or TERM=dumb.

Well if TERM=dumb you can't really provide the service. But obeying OCAML_COLOR seems appropriate.