Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the structured-logger crate to the README #548

Merged
merged 1 commit into from
Jun 4, 2023

Conversation

zensh
Copy link
Contributor

@zensh zensh commented Mar 28, 2023

structured-logger is a logging implementation for the log crate that logs structured values either synchronous or asynchronous, as JSON, CBOR, or any other format, into a file, stderr, stdout, or any other destination.

@zensh
Copy link
Contributor Author

zensh commented Mar 28, 2023

I have updated structured-logger iorust/structured-logger@c421ecc for MSRV checking

@zensh
Copy link
Contributor Author

zensh commented Mar 28, 2023

I have updated structured-logger iorust/structured-logger@c421ecc for MSRV checking

Oh! it come from syn-2.0.11

@JohnTitor
Copy link
Member

I don't think it's related to CI failure here. The failure is due to syn v2 which is a transitive dependency of this crate.

@JohnTitor
Copy link
Member

On the other hand, someone may not call your crate as "most popular one" because it's pretty new.
The problem is that we don't have any solid policy around the list and it could be quite long if we accept crates unconditionally.
Maybe we could link to https://crates.io/keywords/log and/or https://crates.io/keywords/logger instead (one of this approach's cons is that this list can contain unrelated crates). I'd like to hear more opinions, cc @rust-lang/crate-maintainers

@KodrAus
Copy link
Contributor

KodrAus commented Apr 10, 2023

I've opened #551 to discuss the future of this list. Either #550 or #543 should deal with that MSRV build failure too. It looks like we've got a merge conflict, but once that's sorted I'll go ahead and merge this one, as it's in-line with our current treatment of the integrations list.

Thanks for working on this @zensh! 🙂

@JohnTitor JohnTitor merged commit 9ae986d into rust-lang:master Jun 4, 2023
EFanZh pushed a commit to EFanZh/log that referenced this pull request Jul 23, 2023
Bumps [tokio](https://github.com/tokio-rs/tokio) from 1.21.2 to 1.22.0.
- [Release notes](https://github.com/tokio-rs/tokio/releases)
- [Commits](tokio-rs/tokio@tokio-1.21.2...tokio-1.22.0)

---
updated-dependencies:
- dependency-name: tokio
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants