-
Notifications
You must be signed in to change notification settings - Fork 15
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
Dependencies: update node-sass #148
base: main
Are you sure you want to change the base?
Conversation
This is just a test to try installing it via `npm` ``` npm install https://github.com/readthedocs/ethical-ad-client\#humitos/node-sass ```
I thought this would be fine, but I found in adding CI tests here that the sass-loader version doesn't like this. However, modern sass-loader is written for webpack 5, and so there are two upgrades there that we'll need to make -- neither are huge, just more than I can tackle ATM. I proposed a different option in #149 -- drop node-sass entirely for Dart sass, which is compiled to pure JS. No more node-gyp! |
I borrowed some ideas from b9e834d#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519 |
CI is added and passing now, in #149 |
We need to distinguish whether or not run this code, depending if it's imported as a module or via `<script>`
This is just a test to try installing it via
npm