Lazily require node-sass

dlmanning / gulp-sass

SASS plugin for gulp

MIT License

1.56k stars 382 forks source link

Lazily require node-sass #748

Closed HoldYourWaffle closed 3 years ago

HoldYourWaffle commented 4 years ago

Closes #747.

Implementation of #747 for your convenience. I'm technically in violation of the contributor guidelines by opening this PR without prior approval, but I need this functionality right now and I want to use my fork in the meantime.

I'm not sure how I should update the changelog, should I add a new version "heading"?

ncheikh commented 4 years ago

Our company also would like to see this merged as it should allow for node 12 support with the node-sass dependency.

harshjv commented 4 years ago

@dlmanning can you please check & merge this PR?

hasghari commented 3 years ago

Can I help with anything to speed this along? Without this change, we are blocked on Node 14.

b1rdex commented 3 years ago

I've just found https://www.npmjs.com/package/gulp-dart-sass that can be used as a drop-in replacement.

xzyfer commented 3 years ago

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