Sass @imports don't require quotes

[jkbrzt]

For example, this a valid .sass import:

@import partials/layout

But it doesn't get matched by the import reqex:

 /^\s*@import\s+['"](.*)['"]/

It should be something like:

/^\s*@import\s+['"]?(.*?)['"]?/

[es128]

Really? I don't see any example like that in the sass documentation: http://sass-lang.com/docs/yardoc/file.SASS_REFERENCE.html#import.

I'll try it out though, and if I can validate that works I'll tweak the regex.

[es128]

Nope

29 Jul 11:11:41 - error: Compiling of 'vendor/styles/parent.scss' failed. Syntax error: Invalid CSS after "@import ": expected file to import (string or url()), was "_children/layout;"
        on line 7 of standard input
  Use --trace for backtrace.

Although this is true of stylus, and I will be fixing it there.

[jkbrzt]

Yep ;) There is a difference between .scss and .sass (the indented syntax).

Please see the docs:

@import The @import directive in Sass does not require quotes, although they may be used. For example, this SCSS:

@import "themes/dark";
@import "font.sass";

would be this Sass:

@import themes/dark
@import font.sass

[jkbrzt]

I didn't notice your first comment: Yes, it's difficult to find in the docs but the quotes are really optional (link in my previous comment).

[es128]

Ok I was trying it in .scss.