Deprecation of node-sass
See original GitHub issueWhere to find the issue
https://github.com/corona-warn-app/cwa-website/blob/master/package.json https://github.com/corona-warn-app/cwa-website/blob/master/gulpfile.js
Describe the issue
According to https://www.npmjs.com/package/node-sass “Node Sass” used by cwa-website is deprecated:
“Warning: LibSass and Node Sass are deprecated. While they will continue to receive maintenance releases indefinitely, there are no plans to add additional features or compatibility with any new CSS or Sass features. Projects that still use it should move onto Dart Sass.”
Suggested change
Migrate from node sass to Dart Sass see https://sass-lang.com/blog/libsass-is-deprecated which says:
“If you’re a user of Node Sass, migrating to Dart Sass is straightforward: just replace node-sass in your package.json file with sass.”
however the change is not quite so simple. To remediate
Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.
More info and automated migrator: https://sass-lang.com/d/slash-div
it is necessary to run the migrator.
npm uninstall node-sass
npm install sass@latest
- Replace
node-sass
bysass
ingulpfile.js
npm install -g sass-migrator
sass-migrator division **/*.scss
Internal Tracking ID: EXPOSUREAPP-14243
Issue Analytics
- State:
- Created 2 years ago
- Comments:5 (5 by maintainers)
Top GitHub Comments
Not having any intentions of migrating for the moment so this issue will be closed.
@svengabr I agree that @Ein-Tim said it was only a temporary testing issue which he has now resolved. As far as I can see everything is working and there is no urgent need to make any changes in this respect.
If you think that addressing the deprecation of
node-sass
by migrating tosass
is something which might soon be done, then I suggest to leave this issue open as a reminder.If there are no near-term plans to do this, then I suggest closing the issue.