When use indentedSyntax option, have to use .scss to import scss file

[indigofeather]

Hi, I got a problem, when used indentedSyntax option, it occurred error:

ERROR in ./~/css-loader?sourceMap!./~/sass-loader?indentedSyntax&sourceMap&sourceMapContents!./src/css/application.sass
    Module build failed: 
    @import 'foundation/reset'
           ^
          File to import not found or unreadable: foundation/reset.sass
          in /home/**/src/css/application.sass (line 12, column 9)

Before using sass-loader, for example, we have foundation/_reset.scss file, in normal usage (Ex: ruby-sass, node-sass):

// -------------------------------------
//   Foundation
// -------------------------------------

@import 'foundation/reset'
@import 'foundation/helpers'
@import 'foundation/config'
@import 'foundation/base'

use sass-loader, we have to be:

// -------------------------------------
//   Foundation
// -------------------------------------

@import 'foundation/reset.scss'
@import 'foundation/helpers'
@import 'foundation/config'
@import 'foundation/base'

---

But it's not necessarily to set extension for these files.

in index.js @ line 140:

// indentedSyntax is a boolean flag
opt.indentedSyntax = Boolean(opt.indentedSyntax);
fileExt = '.' + (opt.indentedSyntax? 'sass' : 'scss');

in index.js @ line 92:

if (!url.match(extensionMatcher)) {
    extension = contextMatch && contextMatch[0] || fileExt;
    url = url + extension;
}

Without above, it works fine!
Any insight would be appreciated!

[fraina]

+1, it would be awesome if I don't have to check extension before import module's stylesheets.

e.g.

@import './node_modules/susy/sass/susy'
@import './node_modules/breakpoint-sass/stylesheets/breakpoint'
@import 'foundation/susy_config'

instead of

@import './node_modules/susy/sass/susy.scss'
@import './node_modules/breakpoint-sass/stylesheets/breakpoint.scss'
@import 'foundation/susy_config'

[jorrit]

I am assuming you have scss and sass in your resolve.extensions of the webpack.config.js file?

[indigofeather]

Hello, @jorrit
I just use ['', '.js'] in resolve.extensions, and loaders has:

  {
    test: /\.scss$|\.sass$/,
    loader: ExtractTextPlugin.extract('css-loader?sourceMap!autoprefixer-loader?{browsers:["last 2 version"]}!sass-loader?indentedSyntax&sourceMap&sourceMapContents')
  }

[jhnns]

Could you guys check out the current master branch and check if it works for you? I've heavily refactored the import mechanism and need more tests.

[indigofeather]

Hello @jhnns
I've tested the current master branch, it works!

[jhnns]

Thx. That's what I wanted to hear 👍 😀.

[indigofeather]

Waiting for the next version 👍

[jhnns]

Shipped with 2.0.0