Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

use a single bytes.Reader for frame parsing #3536

Merged
merged 1 commit into from
Sep 1, 2022

Conversation

marten-seemann
Copy link
Member

Part of #3526. Depends on #3534.

@marten-seemann marten-seemann changed the title Frame parsing reader use a single bytes.Reader for frame parsing Aug 29, 2022
@codecov
Copy link

codecov bot commented Aug 29, 2022

Codecov Report

Base: 85.48% // Head: 85.48% // No change to project coverage 👍

Coverage data is based on head (0b26365) compared to base (0b26365).
Patch has no changes to coverable lines.

❗ Current head 0b26365 differs from pull request most recent head 1c37ff3. Consider uploading reports for the commit 1c37ff3 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3536   +/-   ##
=======================================
  Coverage   85.48%   85.48%           
=======================================
  Files         138      138           
  Lines       10226    10226           
=======================================
  Hits         8741     8741           
  Misses       1102     1102           
  Partials      383      383           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@marten-seemann marten-seemann added this to the v0.30.0 milestone Aug 30, 2022
@marten-seemann marten-seemann merged commit dfd35cb into master Sep 1, 2022
@marten-seemann marten-seemann deleted the frame-parsing-reader branch September 1, 2022 08:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants