Convert binding to N-API

Author: xzyfer

.travis.yml

@@ -6,28 +6,37 @@ env:
   global:
     - SKIP_SASS_BINARY_DOWNLOAD_FOR_CI=true
 
+stages:
+  - lint
+  - test
+
 jobs:
   include:
+    - stage: lint
+      node_js: "12"
+      os: linux
+      install: npm install --ignore-scripts
+      script: npm run lint || exit 1;
+
+    - stage: test
+      node_js: "13"
+      os: linux
+    - stage: test
+      node_js: "13"
+      os: osx
     - stage: test
       node_js: "12"
       os: linux
-      before_script: npm run lint || exit 1;
       after_success: npm run-script coverage;
-    - stage: platform-test
+    - stage: test
       node_js: "12"
       os: osx
-    - stage: platform-test
+    - stage: test
       node_js: "10"
       os: linux
-    - stage: platform-test
+    - stage: test
       node_js: "10"
       os: osx
-    - stage: platform-test
-      node_js: "13"
-      os: linux
-    - stage: platform-test
-      node_js: "13"
-      os: osx
 
 addons:
   apt:

binding.gyp

@@ -27,15 +27,23 @@
            'SetChecksum': 'true'
         }
       },
+      'cflags!': [ '-fno-exceptions' ],
+      'cflags_cc!': [ '-fno-exceptions' ],
       'xcode_settings': {
         'CLANG_CXX_LANGUAGE_STANDARD': 'c++11',
+        'OTHER_CPLUSPLUSFLAGS': [
+          '-std=c++11'
+        ],
         'CLANG_CXX_LIBRARY': 'libc++',
         'OTHER_LDFLAGS': [],
         'GCC_ENABLE_CPP_EXCEPTIONS': 'NO',
         'MACOSX_DEPLOYMENT_TARGET': '10.7'
       },
+      "dependencies": [
+        '<!(node -p "require(\'node-addon-api\').gyp")'
+      ],
       'include_dirs': [
-        '<!(node -e "require(\'nan\')")',
+        '<!@(node -p "require(\'node-addon-api\').include")',
       ],
       'conditions': [
         ['libsass_ext == "" or libsass_ext == "no"', {

lib/extensions.js

@@ -203,7 +203,12 @@ function getBinaryName() {
     binaryName = [
       platform, '-',
       process.arch, '-',
-      process.versions.modules
+      // Use the lowest support N-API version possible as it gives us
+      // least amount of binary churn. Because N-API versions are additive
+      // We can opt-in to later versions as we choose to adopt new API features.
+      //
+      // See https://nodejs.org/api/n-api.html#n_api_n_api_version_matrix
+      Math.min(process.versions.napi, 3),
     ].join('');
   }
 

package.json

@@ -63,11 +63,13 @@
     "meow": "^5.0.0",
     "mkdirp": "^0.5.1",
     "nan": "^2.14.0",
+    "node-addon-api": "^1.7.1",
     "npmlog": "^4.0.0",
     "request": "^2.88.0",
     "sass-graph": "^3.0.4",
     "stdout-stream": "^1.4.1",
-    "true-case-path": "^2.2.1"
+    "true-case-path": "^2.2.1",
+    "unicode-9.0.0": "^0.7.5"
   },
   "devDependencies": {
     "coveralls": "^3.0.7",