Home Original page

test: remove test-tty-wrap from status file by Trott · Pull Request #28129 · nodejs/node

@nodejs-github-bot added test

Issues and PRs related to the tests.

tty

Issues and PRs related to the tty subsystem.

labels

Jun 8, 2019

@Trott Trott mentioned this pull request

Jun 8, 2019

gireeshpunathil

richardlau

@Trott Trott added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Jun 8, 2019

BridgeAR

@Trott 

The test is believed to no longer be unreliable on AIX. Remove the flaky
designation from the appropriate status file.

Closes: nodejs#9728

PR-URL: nodejs#28129
Fixes: nodejs#9728
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

BridgeAR pushed a commit that referenced this pull request

Jun 17, 2019 
The test is believed to no longer be unreliable on AIX. Remove the flaky
designation from the appropriate status file.

Closes: #9728

PR-URL: #28129
Fixes: #9728
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>