Fix for recent versions of node-sass

[chrislopresto]

This commit contains the relevant subset of changes from
mpalpha@79a8393

Using the latest master commit for node-sass sass/node-sass@11c96ee, I was not getting any CSS output for any SCSS @import, .json or otherwise.

I noticed mpalpha@79a8393 was returning done({ file: url }) instead of sass.NULL. I am not sure why, seeing as the node-sass importer docs say to return NULL in this instance, but I can verify that making this change solved my problem. All imports – SCSS partials and json – are working now.

I also needed to follow suit and remove dist/ from the .gitignore and both /src and /test from the .npmignore to successfully install with npm 3.

For reference, I am using:

• node 5.6.0
• npm 3.7.1
• node-sass latest master sass/node-sass@11c96ee

[pmowrer]

@chrislopresto Sorry, but I can't break from node-sass custom importer guidelines in order to possibly solve your edge case; it may break the importer for other users. If you could provide tests and/or example projects that clearly demonstrate the issue we can revisit.

That said, there's been a lot of confusion and problems around sass.NULL. Perhaps the following threads will help:

sass/node-sass#1356
sass/node-sass#1291

[chrislopresto]

No problem at all. sass/node-sass#1291 sounds like the root cause.

I am packaging node-sass-json-importer as part of an Ember addon that relies on ember-cli-sass, which pulls in node-sass. I'll see if forking ember-cli-sass to use node-sass@beta does the trick.

I'll let you know if that does the trick. Worst case I'll just use my fork until the dust settles.

Cheers.