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

RFC: Set the default log level to INFO #53

Open
Wilfred opened this issue Jan 2, 2025 · 1 comment
Open

RFC: Set the default log level to INFO #53

Wilfred opened this issue Jan 2, 2025 · 1 comment

Comments

@Wilfred
Copy link

Wilfred commented Jan 2, 2025

If I have a test that's emitting a bunch of warnings, it's really useful to see them. I expected that this crate would enable that, but it only seems to log errors (i.e. tracing::error!("foo") but not tracing::warn!("foo")).

I can see them if I set RUST_LOG=warn, but it took me a while to figure it out, and this crate is so close to Just Working out of the box :)

I see this has been requested in #25, but only a per-test setting was added. If it was a crate feature (as the author initially proposed), I'd use it, but maybe test-log needs a higher default log level? This seems to be a common gotcha (e.g. #23 too).

@d-e-s-o
Copy link
Owner

d-e-s-o commented Jan 5, 2025

Yes, I think it makes sense to change, given that output is captured by default anyway. Feel free to open a pull request.

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

No branches or pull requests

2 participants