Add support for CSS and LESS files by default #8

README.md

@@ -40,7 +40,7 @@ See [stylelint options](http://stylelint.io/user-guide/node-api/#options), for t
 
 * `configFile`: You can change the config file location. Default: (`.stylelintrc`)
 * `context`: String indicating the root of your SCSS files. Default inherits from webpack config.
-* `files`: Change the glob pattern for finding files. Default: (`['**/*.s?(a|c)ss']`)
+* `files`: Change the glob pattern for finding files. Default: (`['**/*.{c,sc,sa,le}ss']`)
 * `syntax`: Use `'scss'` to lint .scss files. Default (`undefined`)
 * `formatter`: Use a custom formatter to print errors to the console. Default: (`require('stylelint/dist/formatters/stringFormatter').default`)
 * `failOnError`: Have Webpack's build process die on error. Default: `false`
@@ -54,7 +54,7 @@ module.exports = {
     new StyleLintPlugin({
       configFile: '.stylelintrc',
       context: 'inherits from webpack',
-      files: '**/*.s?(a|c)ss',
+      files: '**/*.{c,sc,sa,le}ss',
       failOnError: false,
     })
   ]

index.js

@@ -39,6 +39,6 @@ function buildOptions(options) {
   }, options, {
     // Default Glob is any directory level of scss and/or sass file,
     // under webpack's context and specificity changed via globbing patterns
-    files: arrify(options.files || '**/*.s?(c|a)ss')
+    files: arrify(options.files || '**/*.{c,sc,sa,le}ss')
   });
 }

test/fixtures/test8/index.js

@@ -0,0 +1,3 @@
+require(getPath('./../../../node_modules/file-loader/index') + '!./test.css!./test.scss!./test.sass!./test.less');
+
+console.log('test8');

test/fixtures/test8/test.css

@@ -0,0 +1,3 @@
+body {
+  foo
+}

test/fixtures/test8/test.less

@@ -0,0 +1,3 @@
+body {
+  foo
+}

test/fixtures/test8/test.sass

@@ -0,0 +1,3 @@
+body {
+  foo
+}

test/fixtures/test8/test.scss

@@ -0,0 +1,3 @@
+body {
+  foo
+}

test/index.js

@@ -120,6 +120,20 @@ describe('sasslint-loader', function () {
     });
   });
 
+  it('should work with CSS, SCSS, SASS and LESS files by default', function (done) {
+    var config = {
+      context: './test/fixtures/test8',
+      entry: './index'
+    };
+
+    pack(assign({}, baseConfig, config), function (err, stats) {
+      expect(err).to.not.exist;
+      expect(stats.compilation.errors.length).to.equal(4);
+      expect(stats.compilation.warnings.length).to.equal(0);
+      done(err);
+    });
+  });
+
   // it('should work with multiple context', function(done) {
   //   var config = {
   //     context: './test/fixtures/test5',