-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Fix logging for rustdoc #75081
Fix logging for rustdoc #75081
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
Going to go ahead and approve this so we get unbroken faster. I suspect that this is the right long-term fix anyway. @bors r+ |
📌 Commit e97e6fb has been approved by |
@@ -43,7 +43,7 @@ extern crate rustc_trait_selection; | |||
extern crate rustc_typeck; | |||
extern crate test as testing; | |||
#[macro_use] | |||
extern crate log; | |||
extern crate tracing as log; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not just change all log
to tracing
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was less work 🤷 feel free to make a follow-up PR
Fix logging for rustdoc rust-lang#74726 (comment)
Fix logging for rustdoc rust-lang#74726 (comment)
Fix logging for rustdoc rust-lang#74726 (comment)
Fix logging for rustdoc rust-lang#74726 (comment)
Rollup of 8 pull requests Successful merges: - rust-lang#74759 (add `unsigned_abs` to signed integers) - rust-lang#75043 (rustc_ast: `(Nested)MetaItem::check_name` -> `has_name`) - rust-lang#75056 (Lint path statements to suggest using drop when the type needs drop) - rust-lang#75081 (Fix logging for rustdoc) - rust-lang#75083 (Do not trigger `unused_braces` for `while let`) - rust-lang#75084 (Stabilize Ident::new_raw) - rust-lang#75103 (Disable building rust-analyzer on riscv64) - rust-lang#75106 (Enable docs on in the x86_64-unknown-linux-musl manifest) Failed merges: r? @ghost
#74726 (comment)