Ability to disable ANSI coloring

[vemv]

ANSI coloring appears to be unconditional:

schema/src/clj/schema/macros.clj

Line 250 in ddb54c8

(error! (utils/format* "Input to %s does not match schema: \n\n\t \033[0;33m %s \033[0m \n\n"

...which causes ugly output in my CIDER test buffers:

...probably this can be problematic as well in various terminal clients.

I'd suggest introducing a JVM property for deciding whether the user wishes coloring.

WDYT?

Thanks - V

[w01fe]

Yeah, that is unfortunate. If the coloring is problematic, I'd maybe prefer to just remove the coloring entirely. PR welcome!

[vemv]

Hi @w01fe !

For the time being I don't have the ability to contribute, due to IP concerns.

I'd appreciate the removal which seems simple enough!

Thanks - V

[philomates]

for context, this behavior was introduced in #383

I find it very useful in situations where it correctly renders the colors, so I would vote for some sort of configuration to toggle it

[vemv]

Yes, I'd be perfectly ok with a JVM property and default to the current behavior

[w01fe]

My potential concerns with the JVM property are that (1) there are many potential ways one might want to configure the printing, and (2) I'm not sure how well that will work in CLJS. A printer that's configurable the same way other options are currently (i.e. a var you can rebind with any print function you like) seems like it might be a better solution to me.

In any case, I agree it should be an easy change, but I'm sorry to say I'm personally not actively developing this library at the moment. Another contributor might decide to pick it up, though.