Merge pull request #364 from ericcornelissen/eslint/no-unused-vars

Update code so "@typescript-eslint/no-unused-vars" passes

.eslintrc.json

@@ -44,7 +44,6 @@
             "@typescript-eslint/no-unsafe-call": "off",
             "@typescript-eslint/no-unsafe-member-access": "off",
             "@typescript-eslint/no-unsafe-return": "off",
-            "@typescript-eslint/no-unused-vars": "off",
             "@typescript-eslint/no-var-requires": "off",
             "@typescript-eslint/prefer-regexp-exec": "off",
             "@typescript-eslint/require-await": "off",

src/config-utils.test.ts

@@ -726,10 +726,7 @@ test("Invalid queries in workflow file handled correctly", async (t) => {
     // This function just needs to be type-correct; it doesn't need to do anything,
     // since we're deliberately passing in invalid data
     const codeQL = setCodeQL({
-      async resolveQueries(
-        _queries: string[],
-        _extraSearchPath: string | undefined
-      ) {
+      async resolveQueries() {
         return {
           byLanguage: {
             javascript: {},