-
Notifications
You must be signed in to change notification settings - Fork 659
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
perf: ionic setup sass #234
Comments
Could you paste the contents from Scanning the node-sass issues, it looks like theres an issue filed about this and Node v0.12 |
I have this issue too, ionic info shows:
Is there a work around, or fix, do you know? |
@toby1kenobi can you post some of the error log as you saw them? It'd help to see that. Or, did you have the same output as the original poster? |
@toby1kenobi and @mifama - Try updating your package.json file - change gulp-sass to be this:
This should bump gulp-sass to use node-sass 2.x, which should resolve that issue. |
Closing - since this is more of a node-sass and gulp-sass issue. |
Sorry for delay, will try your suggestion |
Please do. I was able to recreate it last night on my Windows machine like you had, and bumping that version solved the issue. One other thing to note - at first I had trouble since Python is required, I installed 3.5, that had some issues. I found a stackoverflow post that specified using Python 2.7, which also helped. Not sure if that is absolutely required though. |
Brilliant, that did it - thanks! |
Yep. Were you running python 2.7 or 3.5? |
2.7 - I was anyway, but as a result of some previous node-related situation |
From @mifama on February 6, 2015 23:16
Type: perf
Platform: desktop browser
I updated the ionic version to last version, and now when I do 'ionic setup sass' it's return
Copied from original issue: ionic-team/ionic-framework#3049
The text was updated successfully, but these errors were encountered: