feat: Replace node-sass to sass (#56)

fixes #50 BREAKING CHANGE: Using `sass` instead of `node-sass` package.
vuejs · Apr 11, 2019 · 3e4e3fc · 3e4e3fc
1 parent f3af513
commit 3e4e3fc
Show file tree

Hide file tree

Showing 4 changed files with 105 additions and 258 deletions.
diff --git a/lib/styleProcessors/index.ts b/lib/styleProcessors/index.ts
@@ -21,7 +21,7 @@ const scss: StylePreprocessor = {
     map: any | null,
     options: any
   ): StylePreprocessorResults {
-    const nodeSass = require('node-sass')
+    const nodeSass = require('sass')
     const finalOptions = Object.assign({}, options, {
       data: source,
       file: options.filename,

diff --git a/package.json b/package.json
@@ -42,8 +42,8 @@
     "jest": "^24.0.0",
     "less": "^3.9.0",
     "lint-staged": "^8.1.1",
-    "node-sass": "^4.11.0",
     "pug": "^2.0.3",
+    "sass": "^1.17.3",
     "stylus": "^0.54.5",
     "ts-jest": "^24.0.0",
     "typescript": "^3.3.0",

diff --git a/test/compileStyle.spec.ts b/test/compileStyle.spec.ts
@@ -57,7 +57,7 @@ test('preprocess sass', () => {
   const style = parse({
     source:
       '<style lang="sass">\n' +
-      '$red: rgb(255, 0, 0);\n' +
+      '$red: rgb(255, 0, 0)\n' +
       '.color\n' +
       '   color: $red\n' +
       '</style>\n',