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.

Sign up for GitHub

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

Jump to bottom

Fixes #16823 #16827

Closed

Jarred-Sumner wants to merge 2 commits into main from jarred/fix-write-stream

Collaborator

Jarred-Sumner commented Jan 28, 2025 •

edited

Loading

What does this PR do?

Fixes #16823, probably.

How did you verify your code works?

Need to verify this fixes it.


          Update streams.ts

03e8142

robobun commented Jan 28, 2025 •

edited

Loading

^{Updated 11:26 PM PT - Jan 27th, 2025}

❌ @Jarred-Sumner, your commit 44b23ac has 2 failures in Build #10689:

test/js/web/fetch/fetch.test.ts - 1 failing on 🍎 14 aarch64
test/cli/install/bun-install-registry.test.ts - 2 failing on 🪟 2019 x64-baseline

🧪 try this PR locally:

bunx bun-pr 16827


          Update streams.ts

44b23ac

Collaborator Author

Jarred-Sumner commented Jan 28, 2025

Does not fix the issue.

Jarred-Sumner closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet