search

dlmanning / gulp-sass

SASS plugin for gulp
MIT License
1.56k stars 381 forks source link

Doesn't install under nodejs 16 #797

Closed SagePtr closed 3 years ago

SagePtr commented 3 years ago

gulp-sass requires node-sass@4, but node-sass@4 doesn't support nodejs 16 (different compilation errors). gulp-sass should move to Dart Sass, as the Warning message on https://www.npmjs.com/package/node-sass advices to, or update node-sass dependency to version 6.0.0 if switching is not possible.

vmaurin commented 3 years ago

I think it is not so much about switching the default, but about having the node-sass as mandatory dependency. The project claim to give the option between dart and node-sass, but if you are using dart, you have to download and compile node-sass (exactly what it is failing with node 16).

Personally I would prefer to have neither node-sass nor dart part of the project dependency, even if it is complexifying the installation (the readme will have to state that you must install gulp-sass + one of the sass engine).

new-era-technologies commented 3 years ago

If you would like to use gulp-sass, you need to install node stable and node-sass 5.0.0

Anoesj commented 3 years ago

I'm working on a pull request where all compilers are optional dependencies and node-sass is not installed by default. Will report back when it's finished 🙂