Skip to content

Conversation

@190n
Copy link
Contributor

@190n 190n commented May 19, 2025

What does this PR do?

  • adds test-worker-stack-overflow.js (this is basically fixed since pass test-worker-nested-uncaught.js #19509 but it needed a tiny error message adjustment to pass)
  • tests putting transferable objects in envrionmentData (it should cause spawning any worker to throw unless you put the value in transferList)

How did you verify your code works?

New tests + CI for existing ones

@robobun
Copy link
Collaborator

robobun commented May 19, 2025

Updated 12:40 PM PT - Jul 18th, 2025

@190n, your commit c629aec is building: #20926

@190n 190n requested review from a team and alii and removed request for a team May 20, 2025 22:58
@github-actions github-actions bot requested a review from Electroid as a code owner May 20, 2025 23:59
@alii
Copy link
Member

alii commented May 20, 2025

Approved if CI is green✅

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.

4 participants