Skip to content

test: remove test-gc-http-client from status file - #28130

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:whoopsy2
Jun 10, 2019
Merged

test: remove test-gc-http-client from status file#28130
Trott merged 1 commit into
nodejs:masterfrom
Trott:whoopsy2

Conversation

@Trott

@TrottTrott commented Jun 8, 2019

Copy link
Copy Markdown
Member

test-gc-http-client is no longer believed to be unreliable. Remove it's
entry indicating it's flaky from the status file.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Sadly, an error occurred when I tried to trigger a build. :(

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Jun 8, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 8, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

test-gc-http-client is no longer believed to be unreliable. Remove it's
entry indicating it's flaky from the status file.
Closes: nodejs#22336
PR-URL: nodejs#28130
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.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>
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 64d7056

@Trott
Trott merged commit 64d7056 into nodejs:masterJun 10, 2019
BridgeAR pushed a commit that referenced this pull request Jun 17, 2019
test-gc-http-client is no longer believed to be unreliable. Remove it's
entry indicating it's flaky from the status file.
Closes: #22336
PR-URL: #28130
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.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>
@BridgeARBridgeAR mentioned this pull request Jun 17, 2019
@Trott
Trott deleted the whoopsy2 branch January 13, 2022 22:51
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.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@Trott@nodejs-github-bot@sam-github@jasnell@antsmartian@lpinca@cjihrig@gireeshpunathil@BridgeAR@trivikr