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

Upgrade to hyper version 1 in test framework #6858

Merged
merged 2 commits into from
Sep 26, 2024
Merged

Conversation

Serock3
Copy link
Contributor

@Serock3 Serock3 commented Sep 24, 2024

First step and proof of concept before upgrading hyper in the app repo.
Multiple other crates had to be updates for compatibility.


This change is Reviewable

Copy link

linear bot commented Sep 24, 2024

@Serock3 Serock3 self-assigned this Sep 24, 2024
@Serock3 Serock3 requested a review from dlon September 24, 2024 13:50
@Serock3 Serock3 force-pushed the upgrade-to-hyper-1-des-1207 branch from 3c7b859 to 6aabb53 Compare September 24, 2024 14:22
Copy link

linear bot commented Sep 25, 2024

@Serock3 Serock3 changed the title Upgrade to hyper version 1 Upgrade to hyper version 1 in test framework Sep 25, 2024
@Serock3 Serock3 marked this pull request as ready for review September 25, 2024 07:55
@Serock3 Serock3 force-pushed the upgrade-to-hyper-1-des-1207 branch 3 times, most recently from 9cafbc4 to e9802ea Compare September 25, 2024 14:57
Copy link
Member

@dlon dlon left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 26 files at r1, 21 of 21 files at r2, 3 of 3 files at r3, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@Serock3 Serock3 force-pushed the upgrade-to-hyper-1-des-1207 branch from e9802ea to 6864a62 Compare September 26, 2024 07:02
@Serock3 Serock3 merged commit 8de2c04 into main Sep 26, 2024
40 checks passed
@Serock3 Serock3 deleted the upgrade-to-hyper-1-des-1207 branch September 26, 2024 07:05
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