Skip to content

test: fix typos in test/parallel#45583

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
deokjinkim:221123_test_parallel_typo
Nov 28, 2022
Merged

test: fix typos in test/parallel#45583
nodejs-github-bot merged 1 commit intonodejs:mainfrom
deokjinkim:221123_test_parallel_typo

Conversation

@deokjinkim
Copy link
Contributor

@deokjinkim deokjinkim commented Nov 22, 2022

Fix typos in parallel tests.

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Nov 22, 2022
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Nov 22, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 22, 2022
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Nov 22, 2022

CI: https://ci.nodejs.org/job/node-test-pull-request/48092/

@lpinca lpinca added the fast-track PRs that do not need to wait for 72 hours to land. label Nov 22, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Nov 22, 2022

Fast-track has been requested by @lpinca. Please 👍 to approve.

@ErickWendel
Copy link
Member

ErickWendel commented Nov 23, 2022

I've just scheduled a Rerun for test-asan action as the error is not related to this PR. Let's check if it passes now 🤩

@cola119 cola119 added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 24, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 24, 2022
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Nov 24, 2022

CI: https://ci.nodejs.org/job/node-test-pull-request/48160/

This was referenced Nov 25, 2022
Open
Open
Open
@deokjinkim
Copy link
Contributor Author

deokjinkim commented Nov 27, 2022 •
edited
Loading

@ErickWendel and @cola119, IMO, CI error is not related to this pr. Could you re-run CI when you are available?

@aduh95
Copy link
Contributor

aduh95 commented Nov 27, 2022

CI error is not related to this pr. Could you re-run CI when you are available?

The ASan CI is faulty on your branch (for a reason not related to this PR, indeed), the fix has landed in main so you would need to rebase your branch for the CI to turn green. Would you be able to do that? Let us know if you need help or if you would prefer someone else doing it for you.

@deokjinkim
test: fix typos in test/parallel
162fd7f 
Fix typos in parallel tests.
@deokjinkim deokjinkim force-pushed the 221123_test_parallel_typo branch from ee6ed6c to 162fd7f Compare November 27, 2022 11:40
@deokjinkim
Copy link
Contributor Author

deokjinkim commented Nov 27, 2022

The ASan CI is faulty on your branch (for a reason not related to this PR, indeed), the fix has landed in main so you would need to rebase your branch for the CI to turn green. Would you be able to do that? Let us know if you need help or if you would prefer someone else doing it for you.

Thank you for kind explanation. I rebased this PR.

@aduh95 aduh95 removed the fast-track PRs that do not need to wait for 72 hours to land. label Nov 27, 2022
@aduh95 aduh95 added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 27, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 27, 2022
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Nov 27, 2022

CI: https://ci.nodejs.org/job/node-test-pull-request/48213/

@github-actions github-actions bot mentioned this pull request Nov 28, 2022
Open
20 tasks
@cola119 cola119 added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 28, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 28, 2022
@nodejs-github-bot nodejs-github-bot merged commit 58b60c1 into nodejs:main Nov 28, 2022
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Nov 28, 2022

Landed in 58b60c1

This was referenced Nov 29, 2022
Open
Open
ErickWendel pushed a commit to ErickWendel/node that referenced this pull request Nov 30, 2022
@deokjinkim @ErickWendel
test: fix typos in test/parallel
ffdeefb 
Fix typos in parallel tests.

PR-URL: nodejs#45583
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Erick Wendel <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
@github-actions github-actions bot mentioned this pull request Dec 1, 2022
Open
10 tasks
targos pushed a commit that referenced this pull request Dec 12, 2022
@deokjinkim @targos
test: fix typos in test/parallel
b55bd6e 
Fix typos in parallel tests.

PR-URL: #45583
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Erick Wendel <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
@targos targos mentioned this pull request Dec 12, 2022
Merged
danielleadams pushed a commit that referenced this pull request Dec 30, 2022
@deokjinkim @danielleadams
test: fix typos in test/parallel
4b5c945 
Fix typos in parallel tests.

PR-URL: #45583
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Erick Wendel <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
@danielleadams danielleadams mentioned this pull request Dec 30, 2022
Merged
danielleadams pushed a commit that referenced this pull request Dec 30, 2022
@deokjinkim @danielleadams
test: fix typos in test/parallel
682a71c 
Fix typos in parallel tests.

PR-URL: #45583
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Erick Wendel <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
danielleadams pushed a commit that referenced this pull request Jan 3, 2023
@deokjinkim @danielleadams
test: fix typos in test/parallel
7dbc742 
Fix typos in parallel tests.

PR-URL: #45583
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Erick Wendel <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
danielleadams pushed a commit that referenced this pull request Jan 4, 2023
@deokjinkim @danielleadams
test: fix typos in test/parallel
14372b2 
Fix typos in parallel tests.

PR-URL: #45583
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Erick Wendel <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
danielleadams pushed a commit that referenced this pull request Jan 5, 2023
@deokjinkim @danielleadams
test: fix typos in test/parallel
d3b2252 
Fix typos in parallel tests.

PR-URL: #45583
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Erick Wendel <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Trott Trott Trott approved these changes

@lpinca lpinca lpinca approved these changes

@cjihrig cjihrig cjihrig approved these changes

@richardlau richardlau richardlau approved these changes

@ErickWendel ErickWendel ErickWendel approved these changes

@aduh95 aduh95 aduh95 approved these changes

@cola119 cola119 cola119 approved these changes

@RaisinTen RaisinTen RaisinTen approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@deokjinkim @nodejs-github-bot @ErickWendel @aduh95 @Trott @lpinca @cjihrig @richardlau @cola119 @RaisinTen