Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replace deprecated node-sass with dart sass #104

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

wxw-matt
Copy link

As node-sass is deprecated and it does not work on arm64 (Apple M1), I fix this by using dart sass.

@Ref34t
Copy link

Ref34t commented Nov 1, 2021

Hey @wxw-matt how did you replace node-sass by dart sass

@mariusintlwerft
Copy link

Does it work with node 18 if you just change that? Have you test it? Does it work for you? @wxw-matt

@wxw-matt
Copy link
Author

wxw-matt commented Jun 7, 2022

Does it work with node 18 if you just change that? Have you test it? Does it work for you? @wxw-matt

Hi @virgin71 , I haven't tested it. It should work according to description:

Node.js Compatibility
We consider dropping support for a given version of Node.js to be a breaking change as long as that version is still supported by Node.js. This means that releases listed as Current, Active LTS, or Maintenance LTS according to the Node.js release page. Once a Node.js version is out of LTS, Dart Sass considers itself free to break support if necessary.

https://github.com/sass/dart-sass

@mariusintlwerft
Copy link

Yeah, but adding that one only, will break other dependences.

@wxw-matt
Copy link
Author

wxw-matt commented Jun 7, 2022

Yeah, but adding that one only, will break other dependences.

True, sometimes it happens. Have to fix it one by one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants