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

test: correct test comment #38095

Closed

Conversation

evanlucas
Copy link
Contributor

This corrects a comment on the signal type used in the test.

This corrects a comment on the signal type used in the test.
@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 Apr 5, 2021
@benjamingr
Copy link
Member

Hi @evanlucas 👋 hope life is going great :)

@jasnell jasnell added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 6, 2021
@jasnell
Copy link
Member

jasnell commented Apr 6, 2021

There's no reason to make this one wait the 48 hours to land. Please 👍🏻 to fast-track

@nodejs-github-bot
Copy link
Collaborator

@jasnell jasnell removed the needs-ci PRs that need a full CI run. label Apr 6, 2021
evanlucas added a commit that referenced this pull request Apr 6, 2021
This corrects a comment on the signal type used in the test.

PR-URL: #38095
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@evanlucas
Copy link
Contributor Author

Landed in e155b1f

@evanlucas evanlucas closed this Apr 6, 2021
@evanlucas evanlucas deleted the evanlucas/fix-test-comment branch April 6, 2021 18:34
@marsonya marsonya added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Apr 6, 2021
@marsonya
Copy link
Member

marsonya commented Apr 6, 2021

Sorry about the tag after landing. It was a Pull request filter/search mixup.

targos pushed a commit that referenced this pull request May 1, 2021
This corrects a comment on the signal type used in the test.

PR-URL: #38095
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@danielleadams danielleadams mentioned this pull request May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fast-track PRs that do not need to wait for 48 hours to land. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants