diff options
| author | Rich Trott <rtrott@gmail.com> | 2017-06-19 11:13:30 -0700 |
|---|---|---|
| committer | Myles Borins <mylesborins@google.com> | 2017-07-30 23:11:11 -0500 |
| commit | 910fa50e0ebade03fc0f0c5809c2b5a275679df5 (patch) | |
| tree | 7c76b41077719080d8e9d02985be8692471b5533 | |
| parent | bb74da309c57be80c10336bbf859fb4131dcb7dc (diff) | |
| download | node-new-910fa50e0ebade03fc0f0c5809c2b5a275679df5.tar.gz | |
tools: fix error in custom ESLint rule
Fix previously-unnoticed typo in `required-modules.js`.
Backport-PR-URL: https://github.com/nodejs/node/pull/14360
Refs: https://github.com/nodejs/node/pull/13758#discussion_r122582786
PR-URL: https://github.com/nodejs/node/pull/13758
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
| -rw-r--r-- | tools/eslint-rules/required-modules.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/eslint-rules/required-modules.js b/tools/eslint-rules/required-modules.js index 7691d5dd7d..4a444809b7 100644 --- a/tools/eslint-rules/required-modules.js +++ b/tools/eslint-rules/required-modules.js @@ -75,7 +75,7 @@ module.exports = function(context) { if (foundModules.length < requiredModules.length) { var missingModules = requiredModules.filter( function(module) { - return foundModules.indexOf(module === -1); + return foundModules.indexOf(module) === -1; } ); missingModules.forEach(function(moduleName) { |