Skip to content

Conversation

@Kadrian
Copy link

@Kadrian Kadrian commented Jan 11, 2016

This addresses #187

@jhnns Any objections? 😁

@jhnns
Copy link
Member

jhnns commented Jan 17, 2016

Yes 😁

That won't work, because the loader's test regex matches only scss-files. Why do you want to include that option in the README anyway? It's described in node-sass's readme.

@Kadrian
Copy link
Author

Kadrian commented Jan 17, 2016

@jhnns 1) You're right, updated.
2) The reason is that people usually start using webpack, They find out what loaders are, they figure out they need the sass-loader. Now if they want to configure to compile sass instead of scss, they naturally look at configuration options of the sass-loader. Then they don't find that option. They might even start wondering if sass-loader is actually capable of compiling sass and not only scss. #187 proves there are 6 other people who eventually found out about the option, and then actually took the time to search in the issues if there is something about the indentedSyntax. So I'm sure this is just a fraction of the people who are confused. So why make it any harder than it has to be?

@jhnns
Copy link
Member

jhnns commented Jan 18, 2016

Ok, now I do understand you.

I used to have a small chapter about .sass-files, but then I read somewhere that it is deprecated (which is actually not true). I will re-add that small chapter.

However, I encourage you to learn more about node-sass and the underlying dependencies of all webpack loaders. Because loaders are often just connectors between webpack and another tool. So it's not sufficient to just understand the loader.

One thing, for instance, what people need to know is that node-sass is build on libsass, which is a C-port of the famous ruby library. It is not actually Rubysass. However, it tries to stick to it as close as possible.

@jhnns
Copy link
Member

jhnns commented Mar 3, 2016

I think, #196 solves this "issue" as well

@jhnns jhnns closed this Mar 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants