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

hyper-boring: update to hyper 1.0.0 #107

Closed

Conversation

howardjohn
Copy link

@howardjohn howardjohn commented Mar 10, 2023

This PR upgrades hyper-boring to support hyper 1.0.0-rc.3. Most of this is by moving dependencies from hyper core to hyper-util, as much of the functionality we depend on has been removed from the core.

@Noah-Kennedy
Copy link

Thanks for doing this!

We'll have to wait until hyper 1.0 is fully released to merge and release this, but as soon as that happens I'm happy to cut a release.

@howardjohn howardjohn force-pushed the hyper-boring/adopt-hyper-1.0.0 branch 2 times, most recently from 39798e9 to 5185311 Compare August 17, 2023 14:16
@howardjohn howardjohn force-pushed the hyper-boring/adopt-hyper-1.0.0 branch from 5185311 to 969cb33 Compare November 15, 2023 18:19
@howardjohn
Copy link
Author

Hyper 1.0 is out, so this is almost ready to go. However, hyper-util is broken until hyperium/hyper-util#47 is merged

@howardjohn howardjohn force-pushed the hyper-boring/adopt-hyper-1.0.0 branch 2 times, most recently from d8e829c to 2579aaa Compare November 15, 2023 20:53
@howardjohn howardjohn marked this pull request as ready for review November 15, 2023 20:53
Cargo.toml Outdated
hyper = { version = "0.14", default-features = false }
http = "1.0"
hyper = { version = "1.0", default-features = false }
hyper-util = { git = "https://github.com/hyperium/hyper-util" }
Copy link
Contributor

Choose a reason for hiding this comment

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

This adds a dependency on a not-released crate. Personally, I feel like the update to hyper 1.0 should wait for a release of hyper-util

Copy link
Author

Choose a reason for hiding this comment

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

hyper-util is now published, moved to official release

Copy link
Contributor

Choose a reason for hiding this comment

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

awesome!

@howardjohn howardjohn force-pushed the hyper-boring/adopt-hyper-1.0.0 branch from 2579aaa to 423c260 Compare November 17, 2023 00:13
@howardjohn howardjohn force-pushed the hyper-boring/adopt-hyper-1.0.0 branch from 423c260 to 5185311 Compare November 27, 2023 16:00
@howardjohn
Copy link
Author

Unfortunately I messed this PR up and needed to reopen as #194

@howardjohn
Copy link
Author

superseded by #194

@howardjohn howardjohn closed this Nov 28, 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.

3 participants