3 ms·
Thank you for the suggestions and code review via blog post ;) You are correct that `#[deny]` is what I want with Clippy. The env_logger builder is something
by p4lindromica 8y ago
Thank you for the suggestions and code review via blog post ;)
You are correct that `#[deny]` is what I want with Clippy.
The env_logger builder is something I didn't know about. The homepage for the docs does not even mention it - https://docs.rs/env_logger/0.6.0/env_logger/ https://docs.rs/env_logger/0.6.0/env_logger/. I agree that I should depend on my own env variable, but should cargo not do the same?
- Arnavion 8y agoWell cargo and rustc are both first-party Rust tools, so it's expected that they would use "RUST_LOG". I have wondered why env_logger defaults to using it instead of requiring you to specify which env var you want to parse, but it is what it is.
- p4lindromica 8y agoThank you for sharing your knowledge with me Arnavion. I've applied what I've learned here: https://github.com/lopopolo/punchtop/pull/1 https://github.com/lopopolo/punchtop/pull/1 :)