-
Notifications
You must be signed in to change notification settings - Fork 30k
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: cleanup test-cluster-disconnect-unshared-tcp #8598
test: cleanup test-cluster-disconnect-unshared-tcp #8598
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@RachBLondon May I kindly ask you to reference specific test in the commit message? Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Commit title should be fixed before landing
@RachBLondon no worries. I'll start landing this and will fix the commit title. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
linux-ppc test actually succeeded, the just status did not correctly update to GitHub. |
We've got three LGTMs, CI green and no objections. Landing. |
Replaced var with const PR-URL: #8598 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ilkka Myller <ilkka.myller@nodefield.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
landed in 0f7fffb Thank you for your contribution, @RachBLondon |
Replaced var with const PR-URL: #8598 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ilkka Myller <ilkka.myller@nodefield.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
make -j4 test
(UNIX), orvcbuild test nosign
(Windows) passesAffected core subsystem(s)
test
Description of change
test/parallel/test-cluster-disconnect-unshared-tcp.js
var
withconst
.