search

Drakulix / simplelog.rs

Simple Logging Facility for Rust
https://docs.rs/simplelog/
Apache License 2.0
423 stars 71 forks source link

`TermLogger` uses hard-coded colors #68

Closed JarrettBillingsley closed 3 years ago

JarrettBillingsley commented 3 years ago

I was hoping they were configurable, but: https://github.com/Drakulix/simplelog.rs/blob/195361ccb8442a2f0a8ea2e40fdce6223d0cf0fa/src/loggers/termlog.rs#L173-L179

I don't think this would be hard to add, so if you like the idea, I could make a pull request.

Drakulix commented 3 years ago

Yes, this could definitely be an option for simplelog's ConfigBuilder.

Just make sure to keep the current settings as defaults, when you make a PR. Also please add a re-export for termcolor::Color, so that users do not need to add the dependency themselves.

JarrettBillingsley commented 3 years ago

/pull/69 :)