Use grunt-sass instead of grunt-contrib-sass

[afifsohaili]

grunt-sass is a lot faster than grunt-sass-contrib that this project is using. It's built on top of a C/C++ implementation of SASS compiler. It lacks support for SASS style, but I don't think we're using that syntax at all in this project, so it should be fine (correct me if I'm wrong!).

run npm install then grunt

From grunt-sass repo:

This task uses libsass which is an experimental Sass compiler in C++. In contrast to the original Ruby compiler, this one is much faster, but is missing some features, though improving quickly. It also doesn't support Compass. Check out grunt-contrib-sass if you prefer something more stable, but slower.

[Dogfalo]

I don't think it works. I've tried using LibSass before but I think it doesn't support the syntax we use in _colors.scss.

See #161

I also checked it out and tested it. It doesn't throw errors, but also didn't compile for me.

[afifsohaili]

Ah I see. I didn't try viewing the file that it compiles. I just ran grunt and it didn't throw any errors and I thought it was good.

Thanks! Closing PR.

[tnguyen14]

@Dogfalo libsass should support the map syntax in colors.scss. In fact, I am using gulp-sass, which uses libsass under the hood and it works fine. See here http://sass-compatibility.github.io/#map_data_type

[Dogfalo]

Interesting, it's possible support was added recently. In which case in not
sure why it wouldn't compile in the pull request.
On Dec 17, 2014 10:34 AM, "Tri Nguyen" notifications@github.com wrote:

@Dogfalo https://github.com/Dogfalo libsass should support the map
syntax in colors.scss. In fact, I am using gulp-sass, which uses libsass
under the hood and it works fine. See here
http://sass-compatibility.github.io/#map_data_type

—
Reply to this email directly or view it on GitHub
#287 (comment).

[tnguyen14]

Perhaps clean out your node_modules and re-install?

[Dogfalo]

I'll try that in a bit.
On Dec 17, 2014 10:44 AM, "Tri Nguyen" notifications@github.com wrote:

Perhaps clean out your node_modules and re-install?

—
Reply to this email directly or view it on GitHub
#287 (comment).

[Dogfalo]

Good call, after deleting all my node_modules, it is now compiling properly!

[afifsohaili]

@Dogfalo if you want, I can create another PR for you on this.

[afifsohaili]

@Dogfalo I see that we are using it already in the recent commits. Cool! Pulling it now.