rust-lang / log

Logging implementation for Rust
https://docs.rs/log
Apache License 2.0
2.16k stars 250 forks source link

What to do about the integration list in the README #551

Closed KodrAus closed 2 months ago

KodrAus commented 1 year ago

In https://github.com/rust-lang/log/pull/548#issuecomment-1486869928 @JohnTitor brought up the broader question of what to do with the list of integrations listed in the README. They're currently presented as "the most popular" implementations out there. Right now the list isn't really curated that way; if there's a new crate that integrates with log then we add it to the list.

The function the list serves is giving users a kick-start to get actual data out of log, which on its own simply pipes your events into oblivion.

Alternative strategies to achieving that function are:

NobodyXu commented 1 year ago

Would it be a good idea to mention tracing_log in the list, for it enables forwarding tracing to log or visa versa?

KodrAus commented 1 year ago

@NobodyXu Definitely :+1: It's a bit surprising we don't mention tracing at all in there.

Thomasdezeeuw commented 1 year ago

Maybe we should just change the wording? For example from:

There are many available implementations to choose from, here are some of the most popular ones:

To:

There are many available implementations to choose from, here are some options:

KodrAus commented 1 year ago

@Thomasdezeeuw that sounds good to me.

Thomasdezeeuw commented 5 months ago

@KodrAus any further changes we want to make?

KodrAus commented 2 months ago

@Thomasdezeeuw I think it's all ok now. Are you happy for this issue to be closed?