Support sass syntax with libsass about sass-brunch

[paulmillr]

The plugin should automatically detect the correct syntax. Something is wrong. The tests are passing though. Would you mind sharing the file with us?

[alp82]

I forgot to mention that i use the native libsass library, the ruby version is not installed.

I created a minimal test for that. My file (throws the above error):

div
   color: red

This works though:

div {
  color: red;
}

[paulmillr]

libsass doesn't support .sass.

[paulmillr]

Ah, sorry — just checked the source code. It seems it actually does support it with the sass2scss addition. We'll need to take a look.

[alp82]

gulp-sass uses libsass too and has no problems with .sass syntax

[tamlyn]

I think node-sass just needs indentedSyntax: true to work https://github.com/sass/node-sass#indentedsyntax

[gregtap]

I also get this issue with 1.8.10. Even with

    sass: {
        mode: "native",
        options: {
            indentedSyntax: true  // or "true"
        }
    }

15 Aug 07:06:48 - error: Compiling of 'ondemand_web/static/css/base.sass' failed. invalid top-level expression

#foobar
    opacity: 0
    position: fixed
    bottom: -150px

[hayesgm]

Fixed this by bumping to node-sass 3.0, which currently isn't supported on the mainline of sass-brunch. You can fork this and bump the version of node-sass, or just add the following version which is a current pull request to this library (see #84).

package.json

...
"dependencies": {
  "node-sass": "^3.3.2",
  "sass-brunch": "git+ssh://[email protected]:leadtrader/sass-brunch.git",
  ...
}

EDIT

See #86, I would suggest using the branch hayesgm/sass-brunch#node-sass-v3:

package.json

...
"dependencies": {
  "node-sass": "^3.3.2",
  "sass-brunch": "hayesgm/sass-brunch#node-sass-v3",
  ...
}

[realistschuckle]

Looks like someone can close this issue due to 347c95e.

Just tested it with the current release and it compiles the offending SASS as reported by @coulix.

[es128]

Thanks @realistschuckle!