Skip to content

Conversation

@djc
Copy link
Contributor

@djc djc commented Dec 30, 2025

No description provided.

@djc djc requested review from ChrisDenton and rami3l December 30, 2025 17:30
@ChrisDenton
Copy link
Member

Looking at the lockfile it seems we are using two different versions of reqwest after this? Also, that is a lot more dependencies but maybe that can't be helped.

@djc
Copy link
Contributor Author

djc commented Dec 30, 2025

Looking at the lockfile it seems we are using two different versions of reqwest after this? Also, that is a lot more dependencies but maybe that can't be helped.

What do you mean "a lot more dependencies"? That's not what I'm seeing in the Cargo.lock file.

Yes, some of the opentelemetry crates are still using reqwest 0.12. I was actually working on a PR for that project but the otel feature is not enabled by default anyway (I'm still in favor of just removing it).

@djc djc added this pull request to the merge queue Dec 30, 2025
Merged via the queue into main with commit de2b979 Dec 30, 2025
29 checks passed
@djc djc deleted the reqwest-0.13 branch December 30, 2025 22:53
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.

3 participants