chore: lint unused catch bindings

Author: Renegade334

components/git/land.js

@@ -181,7 +181,7 @@ async function main(state, argv, cli, dir) {
   }
   try {
     session.restore();
-  } catch (err) { // JSON error?
+  } catch { // JSON error?
     if (state === ABORT) {
       await session.abort();
       return;

eslint.config.js

@@ -39,6 +39,12 @@ export default [
         { ignoreRegExpLiterals: true, ignoreUrls: true },
       ],
       '@stylistic/object-property-newline': 'off',
+      'no-unused-vars': ['error', {
+        args: 'none',
+        caughtErrors: 'all',
+        ignoreRestSiblings: true,
+        vars: 'all',
+      }],
       'promise/always-return': ['error', { ignoreLastCallback: true }],
       'n/no-process-exit': 'off',
       'n/no-unsupported-features/node-builtins': 'off',

lib/auth.js

@@ -76,7 +76,7 @@ async function auth(
       let token;
       try {
         ({ username, token } = getMergedConfig());
-      } catch (e) {
+      } catch {
         // Ignore error and prompt
       }
 
@@ -127,7 +127,7 @@ async function auth(
       let token;
       try {
         token = await encryptValue(credentials.token);
-      } catch (err) {
+      } catch {
         console.warn('Failed encrypt token, storing unencrypted instead');
         token = credentials.token;
       }

lib/backport_session.js

@@ -27,7 +27,7 @@ export default class BackportSession extends Session {
       logs = runSync('git',
         ['log', `-${num}`, '--format=%h', rev, '--', file]
       ).trim();
-    } catch (e) {
+    } catch {
       return null;
     }
     if (!logs) {
@@ -282,7 +282,7 @@ export default class BackportSession extends Session {
         ['rev-parse', '--verify', '--quiet', branch]
       );
       return true;
-    } catch (e) {
+    } catch {
       return false;
     }
   }

lib/ci/run_ci.js

@@ -38,7 +38,7 @@ export class RunPRJob {
     try {
       const { crumb } = await this.request.json(CI_CRUMB_URL);
       return crumb;
-    } catch (e) {
+    } catch {
       return false;
     }
   }

lib/landing_session.js

@@ -286,7 +286,7 @@ export default class LandingSession extends Session {
             }
           });
         return this.final();
-      } catch (e) {
+      } catch {
         await runAsync('git', ['rebase', '--abort']);
         const count = subjects.length;
         cli.log(`Couldn't rebase ${count} commits in the PR automatically`);

lib/prepare_security.js

@@ -309,7 +309,7 @@ export default class PrepareSecurityRelease extends SecurityRelease {
         return body.substring(index);
       }
       return body;
-    } catch (error) {
+    } catch {
       this.cli.error(`Could not retrieve the security issue template from ${url}`);
     }
   }

lib/security-release/security-release.js

@@ -121,7 +121,7 @@ export async function commitAndPushVulnerabilitiesJSON(
       ['push', '-u', 'origin', NEXT_SECURITY_RELEASE_BRANCH],
       `This pushes the security release branch to origin/${NEXT_SECURITY_RELEASE_BRANCH}.`
     );
-  } catch (error) {
+  } catch {
     cli.warn('Rebasing...');
     // try to pull rebase and push again
     await runSecurityGitCommand(

lib/security_blog.js

@@ -362,7 +362,7 @@ export default class SecurityBlog extends SecurityRelease {
       const date = await this.promptAnnouncementDate(cli);
       validateDate(date);
       return new Date(date).toISOString();
-    } catch (error) {
+    } catch {
       return PLACEHOLDERS.annoucementDate;
     }
   }

lib/session.js

@@ -160,7 +160,7 @@ export default class Session {
     let sess;
     try {
       sess = this.session;
-    } catch (err) {
+    } catch {
       return fs.rmSync(this.sessionPath, { recursive: true, force: true });
     }