Skip to content

Conversation

@sam-github
Copy link
Contributor

@sam-github sam-github commented Jul 4, 2019

See: nodejs/build#1820

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

@nodejs-github-bot nodejs-github-bot added test Issues and PRs related to the tests. tty Issues and PRs related to the tty subsystem. labels Jul 4, 2019
@nodejs-github-bot
Copy link
Collaborator

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, we need to get builds back to green and we'll work to re-enable once we have a solution.

@mhdawson
Copy link
Member

mhdawson commented Jul 4, 2019

I'd like to suggest fast tracking this.

@mhdawson mhdawson added the fast-track PRs that do not need to wait for 48 hours to land. label Jul 4, 2019
@sam-github
Copy link
Contributor Author

Landed in 8619b19

@sam-github sam-github closed this Jul 4, 2019
@sam-github sam-github deleted the skip-console-tests-on-aix branch July 4, 2019 21:49
sam-github added a commit that referenced this pull request Jul 4, 2019
See: nodejs/build#1820

PR-URL: #28541
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
targos pushed a commit that referenced this pull request Jul 20, 2019
See: nodejs/build#1820

PR-URL: #28541
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
This was referenced Jul 23, 2019
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. tty Issues and PRs related to the tty subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants