Skip to content
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

Allow use with sass-rails master which supports sass 3.3 #160

Merged
merged 1 commit into from
May 17, 2014

Conversation

twalpole
Copy link
Contributor

@twalpole twalpole commented May 8, 2014

No description provided.

@tommymarshall
Copy link

👍

@adamyonk
Copy link

Thank you!

@matthewd
Copy link

Would it be helpful if sass-rails moved some things around so you could override more specifically? We're already duplicating most of evaluate, for example, from Sprockets... it sure seems sucky that you have to reproduce it "from scratch" yet again. 😕

Then again, you don't currently depend on sass-rails, so maybe not.

craigmcnamara added a commit that referenced this pull request May 17, 2014
Allow use with sass-rails master which supports sass 3.3
@craigmcnamara craigmcnamara merged commit f97a4f4 into Compass:master May 17, 2014
@adamyonk
Copy link

So what are the correct versions of sass-rails and compass-rails to use now to get sass up to > 3.3.0?

@twalpole
Copy link
Contributor Author

@adamyonk master branch of both

@adamyonk
Copy link

In that case, compass 0.12.2 keeps barking at me about LoadError: cannot load such file -- sass/script/node (gist). Is there a different version of that that I need?

@adamyonk
Copy link

chriseppstein/compass (master) brings up some issues around sprockets includes, ended up getting everything working with this hack.

@DavidBennettPIO
Copy link

Thanks @adamyonk that fixed it for me also :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants