search

dlmanning / gulp-sass

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

Require `node-sass` on first access to gulpSass.compiler #791

Closed duzun closed 3 years ago

duzun commented 3 years ago

I want to use this plugin with sass, not node-sass, but the line with require('node-sass') throws an error, before I have the chance to gulpSass.compiler = require('sass');. The error message says: Node Sass could not find a binding for your current environment: Linux 64-bit with Node.js 14.x

This PR should fix the issue transparently.

I would also prefer not to include the node-sass lib as a dependency, maybe a peer-dependency.

xzyfer commented 3 years ago

Thanks all, close this to focus efforts on #802 which is more rounded at this point in time.