-
Notifications
You must be signed in to change notification settings - Fork 331
[WIP] SassC support #349
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
[WIP] SassC support #349
Conversation
r? @schneems (@rails-bot has picked a reviewer for you, use r? to override) |
👍 |
@bolandrm I like the approach you're following in this. We can add sassc gem commented on rails Gemfile and then the users can un-comment it to enable as alternative to default sass. |
I've added |
Thanks @guilleiguaran. Testing this in a rails app. Working out some load path issues. |
I hope it will be merge 👍 @bolandrm Will you continue this PR ❓ Or, we should use https://github.com/sass/sassc-rails ❓ |
sassc-rails works fine, but would like sassc in rails by default |
I had some issues with windows and sassc. https://github.com/rails/sprockets/blob/master/test/shared_sass_tests.rb#L117 I tried to fix it, even bought a windows machine. I got hung up on the compilation toolchain and had to move to something else. I'm interested in getting sassc support in sass-rails, would love to help move this along if possible. |
I think sassc had a patch to fix the windows install problems ( sass/sassc#206 ) :D I'm not sure the state of this branch, but I'd love to see this pushed out to all the gems still using sass :) |
See #424 |
This is very much WIP. Comments on the approach are welcome.
This depends on sprockets master (rails/sprockets#148)