#2304 mention electron caveat in readme

[Anonymous]

Originally created by: burningTyger
Originally owned by: saper

Neither bug nor feature just a friendly request to add a caveat to the readme that node-sass doesn't work in electron. Would have saved me a couple of hours of research since I never assumed to google the usual error message with electron in the search string. I was totally not expecting a connection since my app is vue component based and compiles just fine with node-sass (before running under electron though).

Thank you

• NPM version (npm -v): -
• Node version (node -v): -
• Node Process (node -p process.versions): -
• Node Platform (node -p process.platform): -
• Node architecture (node -p process.arch): -
• node-sass version (node -p "require('node-sass').info"): -
• npm node-sass versions (npm ls node-sass): -

[Anonymous]

Originally posted by: mubaidr

I don't think Electron has anything to do with this package. Your blunder script should be responsible to handle the sass files and compiles them to css to be able to load in browsers (or Electron.js)

I am using web-pack (similar to vue-cli generated project) to load vue app (containing sass) inside electron and it works fine.

Or maybe i misunderstood your question?

[Anonymous]

Originally posted by: saper

I think the problem is that people try to use electron node to run node-sass, which will not work. Yes, the generated CSS should be portable.