Custom webpack.config linting

[stewartduffy]

Has anyone had any using eslint-loader in the custom webpack.config? I have the following set up, however eslint-loader is not running

const path = require('path')

module.exports = {
  module: {
    preLoaders: [
      {
        test: /\.js$/,
        loader: 'eslint-loader',
        include: path.resolve(__dirname, '../'),
        exclude: path.resolve(__dirname, '../node_modules/')
      }
    ],
    loaders: [
      {
        test: /\.css?$/,
        loaders: [ 'style', 'raw' ],
        include: path.resolve(__dirname, '../')
      }
    ]
  }
}

[arunoda]

May be it cant read the .eslintrc this way. See if you can can pass that in a different way.

[psimyn]

@stewartduffy you can try passing the .eslintrc location explicitly

module.exports = {
  eslint: {
    configFile: '../.eslintrc'
  },
  module: {
    preLoaders: [
      {
        test: /\.js$/,
        loader: 'eslint-loader',
        include: path.resolve(__dirname, '../'),
        exclude: path.resolve(__dirname, '../node_modules/')
      }
    ],
  etc...

[stewartduffy]

OK so looks like it is an issue with webpack preLoaders rather than eslint-loader.
This config works:

const path = require('path')

module.exports = {
  module: {
    loaders: [
      {
        test: /\.js$/,
        loaders: [ 'eslint-loader' ],
        include: path.resolve(__dirname, '../components/'),
        exclude: /node_modules/
      },
      {
        test: /\.css?$/,
        loaders: [ 'style', 'raw', 'eslint-loader' ],
        include: path.resolve(__dirname, '../')
      }
    ]
  }
}

this config does not work:

const path = require('path')

module.exports = {
  module: {
    preLoaders: [
      {
        test: /\.js$/,
        loaders: [ 'eslint-loader' ],
        include: path.resolve(__dirname, '../components/'),
        exclude: /node_modules/
      }
    ],
    loaders: [
      {
        test: /\.css?$/,
        loaders: [ 'style', 'raw', 'eslint-loader' ],
        include: path.resolve(__dirname, '../')
      }
    ]
  }
}

[stewartduffy]

Looked into this more, can confirm that ti does not support preLoaders. Have made a PR #107 to add support for this.