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: Fixxed tests 580 and 581 for the hyper backend #8707

Closed
wants to merge 1 commit into from

Conversation

liamwarfield
Copy link
Contributor

@liamwarfield liamwarfield commented Apr 14, 2022

Hyper will be able to preserve header order once this PR merges:
hyperium/hyper#2798
This commit adds a few lines setting the connection options
for this feature and enable tests 580 and 851.

Related to issue #8617

DO NOT MERGE UNTIL THE HYPER COMMIT MERGES

Hyper will have be able to preserve header order once this PR merges:
hyperium/hyper#2798
This commit adds a few lines setting the connection options
for this feature.

Related to issue curl#8617
@bagder bagder added the Hyper label Apr 14, 2022
@bagder
Copy link
Member

@bagder bagder commented Apr 15, 2022

Nice work!

@liamwarfield
Copy link
Contributor Author

@liamwarfield liamwarfield commented Apr 23, 2022

@bagder, the hyper commit has merged. Could you please rerun the checks on this commit?

@bagder
Copy link
Member

@bagder bagder commented Apr 25, 2022

@liamwarfield if you rebase this branch and force-push it, it'll run all the CI again...

@liamwarfield
Copy link
Contributor Author

@liamwarfield liamwarfield commented Apr 25, 2022

@bagder
Copy link
Member

@bagder bagder commented Apr 25, 2022

The hyper build is now green! ✔️

@bagder
Copy link
Member

@bagder bagder commented Apr 25, 2022

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants