-
Notifications
You must be signed in to change notification settings - Fork 60
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
Upgrade http / hyper to 1.0 (axum 0.7) #232
Conversation
Thanks for working on this 💪🏻 I had a branch somewhere else locally checking the necessary changes, but it looks you've made more progress here :) |
bc081b2
to
f84092d
Compare
Signed-off-by: Natsuki Ikeguchi <[email protected]>
f84092d
to
e725b33
Compare
bf54c68
to
6554fe3
Compare
6554fe3
to
d165cfd
Compare
@abdolence Could you please review this when you have time? |
e7067bd
to
d165cfd
Compare
Reviewing it right now, this is great work, @siketyan |
All looks good for now. I'm going to merge this and do a bit of testing next week, so feel free to open more fixes/PRs if something popups. Even if it breaks API, it is better to clean up now, since this will v2. Again, thanks for the PR and swift changes even for the latest updates from me 🚀 |
I need to update docs as well before releasing this. |
This is released as part of |
Supersedes #219, #220, #222, #233