Skip to content

Conversation

BridgeAR
Copy link
Member

This reverts commit 1d1ab76.

Fixes: #20227

When landing #20026 I missed failures on Alpine and I could not find a easy way to detect Alpine builds right now, so I decided to revert this until we find a proper fix.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@BridgeAR BridgeAR added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 23, 2018
@nodejs-github-bot nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Apr 23, 2018
@BridgeAR BridgeAR requested a review from bnoordhuis April 23, 2018 15:35
@BridgeAR
Copy link
Member Author

@BridgeAR BridgeAR mentioned this pull request Apr 23, 2018
4 tasks
@BridgeAR
Copy link
Member Author

Landed in e767aa1

@BridgeAR BridgeAR closed this Apr 23, 2018
BridgeAR added a commit to BridgeAR/node that referenced this pull request Apr 23, 2018
This reverts commit 1d1ab76.

PR-URL: nodejs#20228
Fixes: nodejs#20227
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. fast-track PRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI failures

6 participants