Skip to content

Conversation

@Trott
Copy link
Member

@TrottTrott commented Mar 6, 2021

test-worker-message-port-transfer-terminate was fixed in
25447d8 but the status file was not updated to reflect this.

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Mar 6, 2021
@nodejs-github-bot
Copy link
Collaborator

@PoojaDurgadPoojaDurgad added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 8, 2021
test-worker-message-port-transfer-terminate was fixed in 25447d8 but the status file was not updated to reflect this. PR-URL: nodejs#37633 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@Trott
Copy link
MemberAuthor

Trott commented Mar 8, 2021

Landed in 5cea29b

@TrottTrott merged commit 5cea29b into nodejs:masterMar 8, 2021
@TrottTrott deleted the status-only branch March 8, 2021 17:38
danielleadams pushed a commit that referenced this pull request Mar 16, 2021
test-worker-message-port-transfer-terminate was fixed in 25447d8 but the status file was not updated to reflect this. PR-URL: #37633 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@danielleadamsdanielleadams mentioned this pull request Mar 16, 2021
targos pushed a commit that referenced this pull request May 1, 2021
test-worker-message-port-transfer-terminate was fixed in 25447d8 but the status file was not updated to reflect this. PR-URL: #37633 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.needs-ciPRs that need a full CI run.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@nodejs-github-bot@targos@cjihrig@richardlau@PoojaDurgad