fix: prevent lint all files (#77)

webpack-contrib · Feb 12, 2021 · f57cb8e · f57cb8e
1 parent be0391e
commit f57cb8e
Showing 1 changed file with 18 additions and 17 deletions.
diff --git a/src/index.js b/src/index.js
@@ -98,27 +98,28 @@ class ESLintWebpackPlugin {
         return;
       }
 
-      // Gather Files to lint
-      compilation.hooks.finishModules.tap(ESLINT_PLUGIN, (modules) => {
-        /** @type {string[]} */
-        const files = [];
+      /** @type {string[]} */
+      const files = [];
 
-        // @ts-ignore
-        for (const { resource } of modules) {
-          if (resource) {
-            const [file] = resource.split('?');
-
-            if (
-              file &&
-              !files.includes(file) &&
-              isMatch(file, wanted) &&
-              !isMatch(file, exclude)
-            ) {
-              files.push(file);
-            }
+      // @ts-ignore
+      // Add the file to be linted
+      compilation.hooks.succeedModule.tap(ESLINT_PLUGIN, ({ resource }) => {
+        if (resource) {
+          const [file] = resource.split('?');
+
+          if (
+            file &&
+            !files.includes(file) &&
+            isMatch(file, wanted) &&
+            !isMatch(file, exclude)
+          ) {
+            files.push(file);
           }
         }
+      });
 
+      // Lint all files added
+      compilation.hooks.finishModules.tap(ESLINT_PLUGIN, () => {
         if (files.length > 0) {
           lint(files);
         }