Replace node-sass by sass (Dart Sass) to support NodeJS >= 20 #4818
Conversation
azmy60
left a comment
azmy60
left a comment
There was a problem hiding this comment.
I requested a minor change, but overall this seems good!
| sourceMap: true, | ||
| plugins: [require('postcss-prettify')] | ||
| plugins: [require('postcss-prettify')], | ||
| use: { |
There was a problem hiding this comment.
Could we not silent the deprecations? I'm not sure if it's a good idea, though we should fix them at some point, instead of hiding them like this.
That's not unfortunate at all, I'm actually really glad to see some activity here, and if you're willing to take the even bigger PR, that's even better! Do you have news from Oli? Is he alright? This repo is great but it lacks involvement from maintainers. It'd be really nice if you could review some of the open PRs, merge the good ones and release a new version! I have another pending PR that must be integrated to fix a very annoying issue...! |
2 participants
Fixes #4817