respect userOptions.eslintPath

README.md

@@ -159,7 +159,8 @@ module.exports = {
 
 #### `eslintPath` (default: "eslint")
 
-Path to `eslint` instance that will be used for linting.
+Path to `eslint` instance that will be used for linting.  
+If the `eslintPath` is a folder like a official eslint, or specify a `formatter` option. now you dont have to install `eslint` . 
 
 ```js
 module.exports = {
@@ -171,7 +172,7 @@ module.exports = {
         exclude: /node_modules/,
         loader: "eslint-loader",
         options: {
-          eslintPath: path.join(__dirname, "reusable-eslint-rules.js"),
+          eslintPath: path.join(__dirname, "reusable-eslint"),
         }
       },
     ],

index.js

@@ -150,13 +150,25 @@ module.exports = function(input, map) {
     loaderUtils.getOptions(this)
   )
 
+  var userEslintPath = userOptions.eslintPath
+  var formatter = require("eslint/lib/formatters/stylish")
+
+  if (userEslintPath) {
+    try {
+      formatter = require(userEslintPath + "/lib/formatters/stylish")
+    }
+    catch (e) {
+      formatter = require("eslint/lib/formatters/stylish")
+    }
+  }
+
   var config = assign(
     // loader defaults
     {
-      formatter: require("eslint/lib/formatters/stylish"),
+      formatter: formatter,
       cacheIdentifier: JSON.stringify({
         "eslint-loader": pkg.version,
-        eslint: require(userOptions.eslintPath || "eslint").version,
+        eslint: require(userEslintPath || "eslint").version,
       }),
       eslintPath: "eslint",
     },

test/eslint-path.js

@@ -11,7 +11,7 @@ test.cb("eslint-loader can use another instance of eslint via " +
       entry: "./test/fixtures/good.js",
     },
     {
-      eslintPath: path.join(__dirname, "mock/eslint-mock.js"),
+      eslintPath: path.join(__dirname, "mock/eslint"),
     }
   ),
   function(err, stats) {

test/mock/eslint/lib/formatters/stylish.js

@@ -0,0 +1,3 @@
+module.exports = function(result) {
+  return JSON.stringify(result)
+}