Skip to content

Reland Windows test fixes with proper credit#63244

Open
aduh95 wants to merge 3 commits into
nodejs:mainfrom
aduh95:unflaky-test-process-finalization
Open

Reland Windows test fixes with proper credit#63244
aduh95 wants to merge 3 commits into
nodejs:mainfrom
aduh95:unflaky-test-process-finalization

Conversation

@aduh95
Copy link
Copy Markdown
Contributor

@aduh95 aduh95 commented May 11, 2026

#63209 should not have landed with commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. , this PR reverts the commit that landed and re-land the commit with appropriate author and message.

aduh95 and others added 3 commits May 11, 2026 14:10
There was an assumption that the `WeakRef` would be kept alive but
it seems to not always be true on Windows. This commit makes sure
it's kept alive.

Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
Signed-off-by: inoway46 <inoueyuya416@gmail.com>
@aduh95 aduh95 added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label May 11, 2026
@aduh95 aduh95 added the fast-track PRs that do not need to wait for 72 hours to land. label May 11, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Fast-track has been requested by @aduh95. Please πŸ‘ to approve.

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

Labels

commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. fast-track PRs that do not need to wait for 72 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants