Skip to content

Conversation

@clabby
Copy link
Collaborator

@clabby clabby commented Dec 20, 2025

Closes: #786

Overview

Alternative for #2593, but using tokio's BufReader rather than implementing our own read buffering.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 20, 2025

Deploying monorepo with  Cloudflare Pages  Cloudflare Pages

Latest commit: 474535d
Status: ✅  Deploy successful!
Preview URL: https://9d394f22.monorepo-eu0.pages.dev
Branch Preview URL: https://cl-buf-reader-alt.monorepo-eu0.pages.dev

View logs

@patrick-ogrady patrick-ogrady merged commit e89e651 into main Dec 20, 2025
114 checks passed
@patrick-ogrady patrick-ogrady deleted the cl/buf-reader-alt branch December 20, 2025 20:43
@codecov
Copy link

codecov bot commented Dec 20, 2025

Codecov Report

❌ Patch coverage is 87.03704% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.62%. Comparing base (5895e53) to head (474535d).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
runtime/src/network/tokio.rs 87.03% 7 Missing ⚠️
@@            Coverage Diff             @@
##             main    #2595      +/-   ##
==========================================
- Coverage   92.64%   92.62%   -0.02%     
==========================================
  Files         353      353              
  Lines      102339   102534     +195     
==========================================
+ Hits        94810    94977     +167     
- Misses       7529     7557      +28     
Files with missing lines Coverage Δ
runtime/src/lib.rs 97.16% <ø> (-0.07%) ⬇️
runtime/src/network/tokio.rs 85.55% <87.03%> (+0.39%) ⬆️

... and 4 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5895e53...474535d. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

[runtume/stream] Consider using tokio::io::BufReader to wrap tokio::net::TcpStream

3 participants