eslint-loader - npm Package Compare versions

Comparing version 2.1.1 to 2.1.2

CHANGELOG.md

@@ -0,1 +1,6 @@
+# 2.1.2 - 2019-01-31
+
+- [Fix autofix for eslint < 5](https://github.com/webpack-contrib/eslint-loader/commit/d76372676b8badf57df3d6a22be5739d19e25637) by @eschablowski
+
+
 # 2.1.1 - 2018-09-19
@@ -2,0 +7,0 @@

index.js

 "use strict";
 
+//var fs = require("fs");
 var assign = require("object-assign");
@@ -73,3 +74,8 @@ var loaderUtils = require("loader-utils");
     // if enabled, use eslint auto-fixing where possible
-    if (config.fix && (res.results[0].fixableErrorCount > 0 || res.results[0].fixableWarningCount)) {
+    if (
+      config.fix &&
+      (res.results[0].output !== res.src ||
+        res.results[0].fixableErrorCount > 0 ||
+        res.results[0].fixableWarningCount > 0)
+    ) {
       var eslint = require(config.eslintPath);
@@ -241,3 +247,7 @@ eslint.CLIEngine.outputFixes(res);
         try {
-          printLinterOutput(res || {}, config, webpack);
+          printLinterOutput(
+            assign({}, res || {}, { src: input }),
+            config,
+            webpack
+          );
         } catch (e) {
@@ -244,0 +254,0 @@ err = e;

package.json

 {
   "name": "eslint-loader",
-  "version": "2.1.1",
+  "version": "2.1.2",
   "description": "eslint loader (for webpack)",
@@ -5,0 +5,0 @@ "keywords": [

README.md

@@ -33,7 +33,7 @@ # eslint-loader [![Build Status](https://travis-ci.org/webpack-contrib/eslint-loader.svg?branch=master)](https://travis-ci.org/webpack-contrib/eslint-loader)
         }
-      },
-    ],
-  },
+      }
+    ]
+  }
   // ...
-}
+};
 ```
@@ -52,11 +52,8 @@
         exclude: /node_modules/,
-        use: [
-          "babel-loader",
-          "eslint-loader",
-        ],
-      },
-    ],
-  },
+        use: ["babel-loader", "eslint-loader"]
+      }
+    ]
+  }
   // ...
-}
+};
 ```
@@ -76,3 +73,3 @@
         exclude: /node_modules/,
-        loader: "eslint-loader",
+        loader: "eslint-loader"
       },
@@ -82,8 +79,8 @@ {
         exclude: /node_modules/,
-        loader: "babel-loader",
-      },
-    ],
-  },
+        loader: "babel-loader"
+      }
+    ]
+  }
   // ...
-}
+};
 ```
@@ -148,9 +145,9 @@
             // DO NOT USE console.*() directly !
-            return "OUTPUT"
+            return "OUTPUT";
           }
         }
-      },
-    ],
-  },
-}
+      }
+    ]
+  }
+};
 ```
@@ -161,3 +158,3 @@
 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` . 
+If the `eslintPath` is a folder like a official eslint, or specify a `formatter` option. now you dont have to install `eslint` .
 
@@ -174,8 +171,8 @@ ```js
         options: {
-          eslintPath: path.join(__dirname, "reusable-eslint"),
+          eslintPath: path.join(__dirname, "reusable-eslint")
         }
-      },
-    ],
-  },
+      }
+    ]
   }
+};
 ```
@@ -203,8 +200,8 @@
         options: {
-          emitError: true,
+          emitError: true
         }
-      },
-    ],
-  },
-}
+      }
+    ]
+  }
+};
 ```
@@ -230,8 +227,8 @@
         options: {
-          quiet: true,
+          quiet: true
         }
-      },
-    ],
-  },
-}
+      }
+    ]
+  }
+};
 ```
@@ -253,8 +250,8 @@
         options: {
-          failOnWarning: true,
+          failOnWarning: true
         }
-      },
-    ],
-  },
-}
+      }
+    ]
+  }
+};
 ```
@@ -276,11 +273,12 @@
         options: {
-          failOnError: true,
+          failOnError: true
         }
-      },
-    ],
-  },
-}
+      }
+    ]
+  }
+};
 ```
 
 ##### `outputReport` (default: `false`)
+
 Write the output of the errors to a file, for example a checkstyle xml file for use for reporting on Jenkins CI
@@ -302,13 +300,12 @@
           outputReport: {
-            filePath: 'checkstyle.xml',
-            formatter: require('eslint/lib/formatters/checkstyle')
+            filePath: "checkstyle.xml",
+            formatter: require("eslint/lib/formatters/checkstyle")
           }
         }
-      },
-    ],
-  },
-}
+      }
+    ]
+  }
+};
 ```
 
-
 ## Gotchas
@@ -315,0 +312,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

23832

increased by1.21%

Lines of code

236

increased by4.42%

Worsened metrics

Number of lines in readme file

321

decreased by-0.93%

Number of medium supply chain risk alerts

1

increased byInfinity%

No dependency changes