Merge pull request #8 from creaminternet/7-remove-node-sass

Removed node-sass and use dart-sass as a default

Author: edwinbos

README.md

@@ -34,11 +34,6 @@ Check `config/themes.json.sample` to get samples.
 Check `config/watcher.json.sample` to get samples.
 - `usePolling` - set this to `true` to successfully watch files over a network (i.e. Docker or Vagrant) or when your watcher dosen't work well. Warning, enabling this option may lead to high CPU utilization! [chokidar docs](https://github.com/paulmillr/chokidar#performance)
 
-## `sass-compiler.json` structure
-You can choose Sass compiler between the default, but [already deprecated](https://github.com/sass/node-sass/issues/2952), `node-sass` or a newer and faster `dart-sass`.
-
-Since the Dart Sass does not have the same set of features as Node Sass, for now we will keep the older version as default.
-
 ## Optional configurations for 3rd party plugins
 You will find sample config files for theses plugins in `vendor/creaminternet/frontools/config` directory.
 * Create [browserSync](https://www.browsersync.io/) configuration

config/sass-compiler.json.sample

@@ -2,7 +2,6 @@ import gulp from 'gulp'
 import path from 'path'
 import gulpIf from 'gulp-if'
 import dartSass from 'sass'
-import nodeSass from 'node-sass'
 import gulpSass from 'gulp-sass'
 import rename from 'gulp-rename'
 import multiDest from './multi-dest.mjs'
@@ -28,7 +27,6 @@ export default function(name, file) {
   const includePaths = theme.includePaths ? theme.includePaths : []
   const postcssConfig = []
   const disableSuffix = theme.disableSuffix || false
-  const sassCompiler = configLoader('sass-compiler.json', false)
 
   configLoader('.browserslistrc')
 
@@ -68,7 +66,7 @@ export default function(name, file) {
       )
     )
     .pipe(gulpIf(!disableMaps, sourcemaps.init()))
-    .pipe(gulpSass(sassCompiler === 'dart-sass' ? dartSass : nodeSass)({ includePaths: includePaths })
+    .pipe(gulpSass(dartSass)({ includePaths: includePaths })
       .on('error', sassError(env.ci || false)))
     .pipe(gulpIf(production, postcss([cssnano()])))
     .pipe(gulpIf(postcssConfig.length, postcss(postcssConfig || [])))