Gulp-sass: Move `node-sass` to optional or peer dependancy

[simon-tma]

Now that using Dart Sass is documented, it'll be nice to be able to not install node-sass.

[xzyfer]

This has been discussed and is not on the road map for the time being.

On Wed., 17 Oct. 2018, 9:43 am simon-tma, [e-mail] wrote:

Now that using Dart Sass is documented, it'll be nice to be able to not
install node-sass.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dlmanning/gulp-sass/issues/714, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWFe_m0uCidLSZoHuR-WErlS7ETjdks5ulmDigaJpZM4XidkR
.

[simon-tma]

So we're forced to have multiple Sass compilers installed if we want to use the official one? That's annoying.

[xzyfer]

You can look at gulp-dart-sass for the time being

[Kocal]

Weird, since we can use node-sass or dart-sass I don't understand why we are forced to have node-sass to be always installed.

It's even documented in the README :confused:

Install

npm install node-sass gulp-sass --save-dev

If we have to install node-sass ourselves, then it should be marked as a peer dependency, not a direct dependency. :thinking: