tools: lint for unused catch bindings

PR-URL: #24079 Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: Wyatt Preul <wpreul@gmail.com>
nodejs · Nov 6, 2018 · 5e6193f · 5e6193f
1 parent 5407690
commit 5e6193f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -17,7 +17,7 @@ Module._findPath = (request, paths, isMain) => {
   if (!r && hacks.includes(request)) {
     try {
       return require.resolve(`./tools/node_modules/${request}`);
-    } catch (err) {
+    } catch {
       return require.resolve(
         `./tools/node_modules/eslint/node_modules/${request}`);
     }
@@ -212,7 +212,7 @@ module.exports = {
     'no-unsafe-finally': 'error',
     'no-unsafe-negation': 'error',
     'no-unused-labels': 'error',
-    'no-unused-vars': ['error', { args: 'none' }],
+    'no-unused-vars': ['error', { args: 'none', caughtErrors: 'all' }],
     'no-use-before-define': ['error', {
       classes: true,
       functions: false,

diff --git a/lib/stream.js b/lib/stream.js
@@ -54,7 +54,7 @@ try {
     // readable-stream.
     Stream._isUint8Array = process.binding('util').isUint8Array;
   }
-} catch (e) {
+} catch (e) { // eslint-disable-line no-unused-vars
 }
 
 if (!Stream._isUint8Array) {
@@ -74,7 +74,7 @@ if (version[0] === 0 && version[1] < 12) {
                                            chunk.byteOffset,
                                            chunk.byteLength);
     };
-  } catch (e) {
+  } catch (e) { // eslint-disable-line no-unused-vars
   }
 
   if (!Stream._uint8ArrayToBuffer) {