-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Node-sass is not working due to node version difference #1387
Comments
Are you running Cypress from source? Or using the npm package? Do you mind providing a reproducible repo? A good majority of our users use webpack without a problem. |
I'm running from the npm package. I think the problem is with node-sass and node version, webpack is fine. I'll see if I can make a minimal repo to reproduce it. |
There is this issue reporter in cypress-vue-unit-test !
…Sent from my iPhone
On Feb 27, 2018, at 21:35, Dai ***@***.***> wrote:
I'm running from the npm package.
I think the problem is with node-sass and node version, webpack is fine. I'll see if I can make a minimal repo to reproduce it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
https://github.com/dz902/cypress-test-tiny Here we go, a demo. Is it possible to use native node instead of bundled node? As I really only need webpack to compile, and cypress should only be responsible for running the compiled test (maybe this is only for my web-development scenario). Any ideas on whether it's possible? |
Oh man I did not know there is a package for vue. I'm using vue. Really love to test vue with cypress, both great. Is this problem solvable? |
Said issue cypress-io/cypress-vue-unit-test#18 |
We likely need to rebuild the native node-sass prior to distributing Cypress - as it comes prepacked with native binaries. |
Is there anything I can do at the time to alleviate the problem? Or maybe there is a hidden option to use native node instead of bundled node? |
Someone suggested in cypress-io/cypress-vue-unit-test#18 (comment) that using Node 8 to install and run Cypress solves the problem |
Cool, downgrading to 8 and reinstalling node-sass solved the problem. For any searchers landed here: I just found that there is a list of pre-compiled binaries, which user could download (number 57_binding is for Node 8) and should be able to make cypress work. A map of number -> platform can be found in This breaks the integrity of the package though, as this has to be done manually by user. Ultimately there will be a hero to create a real node/js based compiler for sass (or replace it altogether), distributing binaries is just ridiculous for a CSS-transpiler. |
I think I know what the problem is - we build OSX + Windows version of Cypress in each applicable operating system. In those builds - we forgot the bump the locally installed node version. So what's happening is that Cypress is using node Linux is correct - because we did bump the node version there. As long as we bump Windows + OSX node version when distributing Cypress binaries everything "will just work". We'll need to release When we originally released version |
Fixed by #1390 |
Released in |
Hello, is there any way to run Cypress on node v9.x? |
Cypress comes bundled with its own node version and there is no way to change that. |
Is there any way to run the install process with Cypress's built-in node version? |
Is this a Feature or Bug?
Bug / Question
Current behavior:
Desired behavior:
Advices are welcome. Thanks!
The text was updated successfully, but these errors were encountered: