Migrate to termcolor - Githubissues

Drakulix / simplelog.rs

Simple Logging Facility for Rust

https://docs.rs/simplelog/

Apache License 2.0

423 stars 71 forks source link

Migrate to termcolor #59

Closed emmabritton closed 4 years ago

emmabritton commented 4 years ago

Change the dependency of term to termcolor now that term is no longer maintained as per https://github.com/Drakulix/simplelog.rs/issues/54.

It seems termcolor is slightly unreliable on macOS and sometimes doesn't colour the output, but I can't reliably reproduce it. It works fine on Windows and Ubuntu.

Also termcolor doesn't throw an error (at least during my tests) if it can't open stdout so I've removed the Option from TermLogger::new()

Fixes #54

coveralls commented 4 years ago

Coverage increased (+0.4%) to 65.833% when pulling 3da1c5f6b1c5269f5dd63e5168ce2d7b08ef47d4 on raybritton:migrate-to-termcolor into f847219418a88db81ee53eb09b436f6590fbf658 on Drakulix:master.

Drakulix commented 4 years ago

Released with 0.8.0