node-sass-asset-functions icon indicating copy to clipboard operation
node-sass-asset-functions copied to clipboard

Make Sass implementation agnostic ( supporting dart-sass )

Open fsubal opened this issue 7 years ago • 4 comments

Resolves: https://github.com/fetch/node-sass-asset-functions/issues/16


I made this library to be sass-implementation agnostic, that means, add support for dart-sass.

With these changes, node-sass-asset-functions now accepts implementation option ( which excepts require('node-sass') or require('sass'), and when absent it will be implicitly node-sass )


For some reason the test is not passing ( Maybe dart-sass cannot read file correctly, even though the path looks correct ), which is now I'm searching why... Help wanted by someone.

fsubal avatar Oct 08 '18 06:10 fsubal

Love this, can we get it merged @koenpunt ?

marlonmarcello avatar Jul 30 '20 16:07 marlonmarcello

hey folks, can this PR get merged in? would be super helpful since node-sass is basically decrepit at this point

ghost avatar Feb 29 '24 01:02 ghost

for those who want to install this now

npm i --save-dev fetch/node-sass-asset-functions#pull/17/head
"devDependencies": {
  "node-sass-asset-functions": "github:fetch/node-sass-asset-functions#pull/17/head",
}

ghost avatar Feb 29 '24 01:02 ghost

I've released a new version to npm that replaces node-sass with sass (Dart Sass). Development will continue in this fork: https://github.com/koenpunt/node-sass-asset-functions

koenpunt avatar Feb 29 '24 18:02 koenpunt