Compile error on a file which does not return CSS about sass-brunch

[es128]

Thanks for identifying the issue and a fix. Please do send a PR. Starting with a failing test would be preferable.

[ricardobanegas]

I still get the pipeline.js problem. Not getting this fix with the latest 1.8.10-version of sass-brunch.

Steps to reproduce:

• npm install sass-brunch --save-dev
• sed -n 97,97p node_modules/sass-brunch/index.js

Results in: if(data.css) data = data.css; should be if('css' in data) data = data.css; according to 9ab5df8

sass-brunch version is 1.8.10

npm ls --depth=0 | grep sass-brunch
├── [email protected]

[sambonbonne]

According to the commits history (https://github.com/brunch/sass-brunch/commits/master), my fix is not in the 1.8.10, you'll need to wait for a new update ... In fact, there is no fix since mine and the 1.8.10 version.
While waiting for a developer answer or a new version, maybe you should apply my fix manually ?

[ricardobanegas]

Oh snap! Thanks for the info, yes already applied the patch but was hoping it was in the update so that I don't need to do an ugly fix for my team.

For others that have the same problem using npm install sass-brunch, you can use bleeding-edge like this in your package.json (not pretty):

"sass-brunch": "https://github.com/brunch/sass-brunch/tarball/master",

[ricardobanegas]

@paulmillr Have you planned a release of sass-brunch with @smumu fix? Thanks!

[es128]

just published 1.8.11 with the fix

[ricardobanegas]

Great thanks.