search

thgh / rollup-plugin-scss

Rollup and compile multiple .scss, .sass and .css imports
MIT License
135 stars 46 forks source link

Update documentation on usage of indented syntax #50

Closed Robin-Hoodie closed 4 years ago

Robin-Hoodie commented 4 years ago

I just spent a couple of hours figuring out why I kept getting syntax errors along the lines of:

Error: expected ";".

I figured out pretty quickly that the plugin was parsing this without using the indented syntax, even though my files were being imported as .sass files, not .scss files.

After digging through the Sass Javascript API options I found the indentedSyntax option which was passed through by the plugin.

I think the documentation should:

  1. Make it more clear that you can use the options of the Sass Javascript API
  2. Especially point out the indentedSyntax option

I'm happy to make these changes to the docs as a PR if needed.

Robin-Hoodie commented 4 years ago

One other thing on this: while looking through the sourcecode I noticed that the include option is being used, though it doesn't seem to be documented either.

thgh commented 4 years ago

PR welcome 😄