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 ureq example to version 3 of the ureq crate. #186

Merged
merged 2 commits into from
Feb 21, 2025

Conversation

jfgoog
Copy link
Contributor

@jfgoog jfgoog commented Feb 20, 2025

No description provided.

@jfgoog jfgoog force-pushed the ureq-3 branch 2 times, most recently from 1253d3a to 5d36eea Compare February 20, 2025 20:19
Copy link
Collaborator

@Byron Byron left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

I have turned the Response conversion back into something more similar to what it was before and hope there was no other reason for not doing that in the first place.

@Byron Byron merged commit 857c063 into frewsxcv:master Feb 21, 2025
3 checks passed
@jfgoog
Copy link
Contributor Author

jfgoog commented Feb 24, 2025

Thanks. Maybe the response conversion in examples/sparse_http_reqwest.rs can be cleaned up similarly? I basically copied it from there.

@Byron
Copy link
Collaborator

Byron commented Feb 25, 2025

Yes, please feel free to open another PR if there is something left to clean. Thanks a lot for your help.

@jfgoog
Copy link
Contributor Author

jfgoog commented Feb 25, 2025

I took a look at the reqwest example. I think it's correct as-is, because reqwest uses its own response type, not the one from the http crate.

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