eleventy-plugin-sass
eleventy-plugin-sass copied to clipboard
Replace node-sass with dart-sass
Closes #26 by switching to the canonical Dart Sass library.
The current implementation (using node-sass
) crashes on macOS Big Sur, which is unsupported by node-sass
.
This PR fixes that.
Any news here?
Was just about to do the same. What can we do to see this merged?
Pinging @Sonaryr, would you be able to please take a look at this? 🙏 This looks like a relatively simple change. 😄
Not that I care all that much (since I don't have any dependencies on the older node-sass
) but it might also make sense to implement a configuration option as well for backward compatibility (i.e. allow user to pass in their preferred compiler). That way maybe it'd see more adoption in case it breaks anything. e.g.
eleventyConfig.addPlugin(require('eleventy-plugin-sass'), {
// ...
compiler: require('node-sass'),
// ...
});
Just created #32 inspired by this pull request which addresses two issues above, i.e.:
- Fixes the dependency issue on
sass
and - Allows you to override via new
compiler
option in case you still wanted to usenode-sass
for some reason.