Drakulix / simplelog.rs

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

Configurable time format #13

Closed djrodgerspryor closed 6 years ago

djrodgerspryor commented 6 years ago

Add the time_format config option which will be used as the format string for writing the time.

I also replaced the time crate with chrono (https://github.com/chronotope/chrono) because it's more well supported.

https://github.com/Drakulix/simplelog.rs/issues/12

coveralls commented 6 years ago

Coverage Status

Coverage increased (+2.5%) to 81.277% when pulling 28a0e473c04ac996fa4faf48b6a229c14f2ff8da on djrodgerspryor:time_format_config into 2eab3de46cf3a670b175cb00ab82b63805d8af4a on Drakulix:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+2.5%) to 81.277% when pulling 28a0e473c04ac996fa4faf48b6a229c14f2ff8da on djrodgerspryor:time_format_config into 2eab3de46cf3a670b175cb00ab82b63805d8af4a on Drakulix:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+2.5%) to 81.277% when pulling 28a0e473c04ac996fa4faf48b6a229c14f2ff8da on djrodgerspryor:time_format_config into 2eab3de46cf3a670b175cb00ab82b63805d8af4a on Drakulix:master.

Drakulix commented 6 years ago

Thanks! This PR looks really good, I welcome the changes.

However because this is a breaking change I will delay the crates.io release until I found the time to tackle #11, which should happen soon.