Skip to content
New issue

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

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

Already on GitHub? # to your account

use tracing instead of log #12973

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

use tracing instead of log #12973

wants to merge 4 commits into from

Conversation

Levi0804
Copy link

This PR addresses #12911. Please let me know if these changes look good.

@Levi0804 Levi0804 requested a review from a team as a code owner February 21, 2025 20:09
@Levi0804 Levi0804 requested a review from saketh-are February 21, 2025 20:09
@@ -27,7 +27,6 @@ enum Commands {

#[tokio::main]
async fn main() -> anyhow::Result<()> {
env_logger::init();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't it necessary to initialize a tracing logger here?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, I'll add that.

@Levi0804 Levi0804 requested a review from nagisa February 22, 2025 07:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants