Skip to content

Conversation

@jasnell
Copy link
Member

@jasnell jasnell commented Apr 17, 2017

Sigh... I think the jet lag is setting in and I need to call it quits for the afternoon. 70b51c8 is broken on Windows.

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

test

@nodejs-github-bot nodejs-github-bot added node-api Issues and PRs related to the Node-API. test Issues and PRs related to the tests. labels Apr 17, 2017
@jasnell
Copy link
Member Author

jasnell commented Apr 17, 2017

jasnell added a commit that referenced this pull request Apr 17, 2017
This reverts commit 70b51c8.

PR-URL: #12475
Reviewed-By: Colin Ihrig <[email protected]>
@jasnell
Copy link
Member Author

jasnell commented Apr 17, 2017

Landed

(edit by @addaleax: in 6c60691)

@jasnell jasnell closed this Apr 17, 2017
@jasnell jasnell mentioned this pull request May 11, 2017
@gibfahn gibfahn mentioned this pull request Jun 15, 2017
3 tasks
gabrielschulhof pushed a commit to gabrielschulhof/node that referenced this pull request Apr 10, 2018
This reverts commit 70b51c8.

PR-URL: nodejs#12475
Reviewed-By: Colin Ihrig <[email protected]>
MylesBorins pushed a commit that referenced this pull request Apr 16, 2018
This reverts commit 70b51c8.

Backport-PR-URL: #19447
PR-URL: #12475
Reviewed-By: Colin Ihrig <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Apr 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

node-api Issues and PRs related to the Node-API. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants