search

Drakulix / simplelog.rs

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

Remove newline from TestLogger time output #114

Closed lazorchakp closed 1 year ago

lazorchakp commented 1 year ago

I noticed a println! where I would have expected a print! in the TestLogger's write_time function. This was causing log statements to be formatted as:

15:28:59
[INFO] [my_project/src/foo.rs:441] Finished Task 1
15:28:59
[INFO] [my_project/src/foo.rs:444] Processing 0 items for Task 2
15:29:01
[INFO] [my_project/src/bar.rs:482] 1 Total Result

With this change, TestLogger output looks like:

15:28:59 [INFO] [my_project/src/foo.rs:441] Finished Task 1
15:28:59 [INFO] [my_project/src/foo.rs:444] Processing 0 items for Task 2
15:29:01 [INFO] [my_project/src/bar.rs:482] 1 Total Result
coveralls commented 1 year ago

Coverage Status

Coverage decreased (-10.6%) to 53.947% when pulling 84810585c101ba7d23eb580730a4f987a09066fc on lazorchakp:testlogger-remove-newline into c927c19b115fd3c1f34dc5c7dde4abd2a45422a0 on Drakulix:master.

Drakulix commented 1 year ago

Good catch! Thanks :)

Shatur commented 1 year ago

@Drakulix could you please draft a new release with this bugfix?

Drakulix commented 1 year ago

@Drakulix could you please draft a new release with this bugfix?

Published as 0.12.1