WIP: Sanity check with sassc gem

[ashmaroli]

CAUTION!
Experimental setup.
Review at your own risk!

Summary

Sanity check with sassc gem

[pathawks]

What is this all about?

[ashmaroli]

What is this all about?

@pathawks I'm just testing script/default-site with your sassc branch at jekyll-sass-converter..
The build fails because of the following line:

jekyll/lib/jekyll/theme.rb

Lines 44 to 45 in 452c1c8

	External.require_with_graceful_fail("sass") unless defined?(Sass)
	Sass.load_paths << sass_path

I did not go into specifics in the opening comment because this PR is not intended to be merged to master

[pathawks]

this PR is not intended to be merged to master

Isn’t that what the R in Pull Request stands for, a request for changes to be merged? I’m being a little bit cheeky, but I’m just trying to understand what action you are looking for with this PR.

[ashmaroli]

I’m being a little bit cheeky

Please do not waste your time with this "branch"..
There are plenty of other "PRs" that would benefit from your feedback..

I’m just trying to understand what action you are looking for with this PR.

My primary intention was to "test" building the "default site" with the sassc-2.0.0 on Travis CI since I do not have access to a UNIX environment. Perhaps, I should've just "experimented" on my own fork instead. (I'll make it a point to do that in the future)..

As evident from the logs, theme.rb will definitely need to be patched to use the sassc gem.., and the sass-converter should ideally handle "load paths from a theme".. but that's best discussed at the plugin's repo..

[pathawks]

Ok, cool 👍🏼

I wasn’t sure if you were trying to imply that there were something I needed to do for The Great Sassc Migration. I follow now.

You could always use a virtual machine 😉