Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Add support for Node 12 #2633

Merged
merged 7 commits into from
Apr 26, 2019
Merged

Add support for Node 12 #2633

merged 7 commits into from
Apr 26, 2019

Conversation

xzyfer
Copy link
Contributor

@xzyfer xzyfer commented Apr 23, 2019

Fixes #2632

@xzyfer
Copy link
Contributor Author

xzyfer commented Apr 23, 2019

Looks like https://github.com/nodejs/nan doesn't yet support Node 12 so we're blocked on compiling Node 12 binaries.

@xzyfer xzyfer mentioned this pull request Apr 23, 2019
@xzyfer
Copy link
Contributor Author

xzyfer commented Apr 24, 2019

Created a tracking issue nodejs/nan#849

@nschonni
Copy link
Contributor

Also will need appveyor/ci#2921

appveyor.yml Outdated Show resolved Hide resolved
.travis.yml Show resolved Hide resolved
.travis.yml Show resolved Hide resolved
@xzyfer
Copy link
Contributor Author

xzyfer commented Apr 26, 2019

Got this green using the workaround from appveyor/ci#2921 (comment). Hopefully this should mean no more waiting for AppVeyor.

@sass sass locked and limited conversation to collaborators Apr 26, 2019
Copy link
Contributor

@nschonni nschonni left a comment

Choose a reason for hiding this comment

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

LGTM

@nschonni
Copy link
Contributor

Might want to update the package.json with this, unless you are going to do that separately before tagging

@xzyfer xzyfer merged commit bc78b22 into sass:master Apr 26, 2019
@xzyfer xzyfer deleted the node-12 branch April 26, 2019 10:17
@xzyfer
Copy link
Contributor Author

xzyfer commented Apr 26, 2019

Bumped package.json on master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

support for node.js 12.x
3 participants