Skip to content

fix: fail pipeToSync when sync writer returns false - #19

Open
trivikr wants to merge 1 commit into
WinterTC55:mainfrom
trivikr:pipe-to-sync-byte-accounting-sync-write-false
Open

trivikr wants to merge 1 commit into
WinterTC55:mainfrom
trivikr:pipe-to-sync-byte-accounting-sync-write-false

Conversation

@trivikr

@trivikr trivikr commented May 31, 2026

Copy link
Copy Markdown
Contributor

Fixes: #16

@jasnell

jasnell commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

@trivikr ... can I ask you to rebase this?

@trivikr
trivikr force-pushed the pipe-to-sync-byte-accounting-sync-write-false branch from fe36398 to 97b03dd Compare October 4, 2026 18:47
@trivikr

trivikr commented Oct 4, 2026

Copy link
Copy Markdown
Contributor Author

can I ask you to rebase this?

Done.

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.

Clarify pipeToSync() byte accounting when SyncWriter.write() returns false

2 participants