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

refactor: use upstream http_body_util::BodyDataStream #1514

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

tottoto
Copy link
Contributor

@tottoto tottoto commented Jun 11, 2024

Uses upstream http_body_util::BodyDataStream introduced at http-body-util 0.1.2.

https://github.com/hyperium/http-body/blob/http-body-util-v0.1.2/http-body-util/CHANGELOG.md.

Signed-off-by: tottoto <tottotodev@gmail.com>
Copy link

codecov bot commented Jun 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.0%. Comparing base (5aa8f83) to head (84afc1c).

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1514     +/-   ##
=======================================
- Coverage   75.0%   75.0%   -0.0%     
=======================================
  Files         78      78             
  Lines       6872    6870      -2     
=======================================
- Hits        5151    5149      -2     
  Misses      1721    1721             
Files Coverage Δ
kube-client/src/client/body.rs 68.8% <ø> (-1.8%) ⬇️

Copy link
Member

@clux clux left a comment

Choose a reason for hiding this comment

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

Freshly released I see :D

Thank you!

@clux clux added this to the 0.92.0 milestone Jun 11, 2024
@clux clux added the changelog-exclude changelog excluded prs label Jun 11, 2024
@clux clux merged commit f8a8407 into kube-rs:main Jun 11, 2024
16 of 17 checks passed
@tottoto tottoto deleted the body-data-stream branch June 11, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-exclude changelog excluded prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants