Commit da10866c by xzyfer

Add support for Node 12

parent 0f86a0ae
......@@ -18,6 +18,12 @@ jobs:
node_js: "node"
os: osx
- stage: platform-test
node_js: "11"
os: linux
- stage: platform-test
node_js: "11"
os: osx
- stage: platform-test
node_js: "10"
os: linux
- stage: platform-test
......
## v4.12.0
https://github.com/sass/node-sass/releases/tag/v4.12.0
## v4.11.0
https://github.com/sass/node-sass/releases/tag/v4.11.0
......
......@@ -4,6 +4,7 @@
NodeJS | Minimum node-sass version | Node Module
--------|--------------------------|------------
Node 12 | 4.12+ | 72
Node 11 | 4.10+ | 67
Node 10 | 4.9+ | 64
Node 8 | 4.5.3+ | 57
......
......@@ -73,6 +73,9 @@
- nodejs_version: 11
GYP_MSVS_VERSION: 2015
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2015
- nodejs_version: 12
GYP_MSVS_VERSION: 2015
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2015
install:
- ps: Install-Product node $env:nodejs_version $env:platform
......@@ -166,7 +169,10 @@
- nodejs_version: 11
GYP_MSVS_VERSION: 2015
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2015
- nodejs_version: 12
GYP_MSVS_VERSION: 2015
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2015
install:
- ps: Install-Product node $env:nodejs_version $env:platform
- node --version
......
......@@ -77,6 +77,7 @@ function getHumanNodeVersion(abi) {
case 59: return 'Node.js 9.x';
case 64: return 'Node.js 10.x';
case 67: return 'Node.js 11.x';
case 72: return 'Node.js 12.x';
default: return false;
}
}
......
......@@ -5,7 +5,7 @@
char* create_string(Nan::MaybeLocal<v8::Value> maybevalue) {
v8::Local<v8::Value> value;
if (maybevalue.ToLocal(&value)) {
if (value->IsNull() || !value->IsString()) {
return 0;
......@@ -14,7 +14,7 @@ char* create_string(Nan::MaybeLocal<v8::Value> maybevalue) {
return 0;
}
v8::String::Utf8Value string(value);
Nan::Utf8String string(value);
char *str = (char *)malloc(string.length() + 1);
strcpy(str, *string);
return str;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment