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

update reqwest-tracing #222

Merged
merged 1 commit into from
Dec 1, 2021
Merged

update reqwest-tracing #222

merged 1 commit into from
Dec 1, 2021

Conversation

Geal
Copy link
Contributor

@Geal Geal commented Nov 30, 2021

@Geal Geal requested review from cecton and garypen November 30, 2021 16:23
@Geal Geal force-pushed the update-reqwest-tracing branch from 43432e1 to 52c32b9 Compare November 30, 2021 16:25
@@ -36,7 +36,7 @@ static_assertions = "1"
test-log = { version = "0.2.8", default-features = false, features = ["trace"] }
tokio = { version = "1", features = ["full"] }
# don't bump to 0.3 until we bump tracing-opentelemetry to 0.16
Copy link
Contributor

Choose a reason for hiding this comment

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

You can ditch the note then XD

@Geal Geal self-assigned this Dec 1, 2021
@Geal
Copy link
Contributor Author

Geal commented Dec 1, 2021

merging this right now, I need it for #224 (I'm working with tracing-subscriber)

@Geal Geal merged commit 4b8ac42 into main Dec 1, 2021
@Geal Geal deleted the update-reqwest-tracing branch December 1, 2021 16:27
tinnou pushed a commit to Netflix-Skunkworks/router that referenced this pull request Oct 16, 2023
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.

2 participants