sfackler / rust-log-panics

Apache License 2.0
52 stars 14 forks source link

Change log target from `log_panics` to `panic` #2

Closed Thomasdezeeuw closed 6 years ago

Thomasdezeeuw commented 6 years ago

Currently when logging it uses the default target, which is the module name log_panics. But in my opinion it would be nicer to change this to panic, since the panic itself really has nothing to do with the module.

2017-08-04T12:32:58.203374+00:00 [ERROR] log_panics: thread 'tests::log_output' panicked at 'oops'
// vs
2017-08-04T12:32:58.203374+00:00 [ERROR] panic: thread 'tests::log_output' panicked at 'oops'

Would a change like that be acceptable?

sfackler commented 6 years ago

Sure, that seems like a fine thing to do.

Thomasdezeeuw commented 6 years ago

Thanks, any change for a minor update to include the change on crates.io?

sfackler commented 6 years ago

Done

Thomasdezeeuw commented 6 years ago

Thanks.