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

Fixes #13816 #13906

Merged
merged 12 commits into from
Nov 2, 2024
Merged

Fixes #13816 #13906

merged 12 commits into from
Nov 2, 2024

Conversation

Jarred-Sumner
Copy link
Collaborator

What does this PR do?

I wanted to go and make some changes to #13840, so here's a slightly revised version with test failures hopefully fixed

Closes #13840
Fixes #13816

How did you verify your code works?

Tests

@robobun
Copy link

robobun commented Sep 12, 2024

@pfgithub, your commit 163f922 has 1 failures in #5577:

  • test/cli/hot/hot.test.ts - timeout on 🪟 2019 x64-baseline
  • @nektro nektro self-assigned this Oct 9, 2024
    @pfgithub pfgithub force-pushed the jarred/resume-webkit-streams-fixes branch 2 times, most recently from 060113d to d8ede81 Compare October 29, 2024 23:51
    pfgithub and others added 10 commits November 1, 2024 12:36
    Ignore ERR_STREAM_RELEASE_LOCK in process.stdin
    
    Reject pending reads when a ReadableStream's lock is released
    
    WebKit/WebKit@f8ae40c
    
    Update the ReadableStream promise resolution ordering on pull rejection
    
    WebKit/WebKit@1f74c15
    
    Fix pipeThrough unhandled rejections on readable errors
    
    WebKit/WebKit@07e4b92
    
    Fix pipeTo unhandled rejections on readable errors
    
    WebKit/WebKit@3a75b5d
    
    Fixes #13816.
    …update the test to wait for stdout rather than waiting 100ms before the first write
    @pfgithub pfgithub force-pushed the jarred/resume-webkit-streams-fixes branch from d8ede81 to 40e0b3c Compare November 1, 2024 21:19
    @Jarred-Sumner Jarred-Sumner merged commit 6914c5e into main Nov 2, 2024
    47 of 49 checks passed
    @Jarred-Sumner Jarred-Sumner deleted the jarred/resume-webkit-streams-fixes branch November 2, 2024 01:38
    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.

    releasing lock of reader whose stream is still in readable state when cancelling a TransformStream
    7 participants