Lint: Enabling 'noUnusedLocals'

src/actions/ApplyBan.ts

@@ -33,7 +33,6 @@ import { redactUserMessagesIn } from "../utils";
 export async function applyUserBans(lists: BanList[], roomIds: string[], mjolnir: Mjolnir): Promise<RoomUpdateError[]> {
     // We can only ban people who are not already banned, and who match the rules.
     const errors: RoomUpdateError[] = [];
-    let bansApplied = 0;
     for (const roomId of roomIds) {
         try {
             // We specifically use sendNotice to avoid having to escape HTML
@@ -76,7 +75,6 @@ export async function applyUserBans(lists: BanList[], roomIds: string[], mjolnir
                                 await logMessage(LogLevel.WARN, "ApplyBan", `Tried to ban ${member.userId} in ${roomId} but Mjolnir is running in no-op mode`, roomId);
                             }
 
-                            bansApplied++;
                             banned = true;
                             break;
                         }

src/commands/KickCommand.ts

@@ -15,19 +15,9 @@ limitations under the License.
 */
 
 import { Mjolnir } from "../Mjolnir";
-import BanList, { RULE_ROOM, RULE_SERVER, RULE_USER, USER_RULE_TYPES } from "../models/BanList";
-import { LogLevel, LogService, MatrixGlob, RichReply } from "matrix-bot-sdk";
-import { RECOMMENDATION_BAN, recommendationToStable } from "../models/ListRule";
+import { LogLevel } from "matrix-bot-sdk";
 import config from "../config";
 import { logMessage } from "../LogProxy";
-import { DEFAULT_LIST_EVENT_TYPE } from "./SetDefaultBanListCommand";
-
-interface Arguments {
-    list: BanList;
-    entity: string;
-    ruleType: string;
-    reason: string;
-}
 
 // !mjolnir kick <user> [room] [reason]
 export async function execKickCommand(roomId: string, event: any, mjolnir: Mjolnir, parts: string[]) {

tsconfig.json

@@ -6,6 +6,7 @@
         "module": "commonjs",
         "moduleResolution": "node",
         "noImplicitReturns": true,
+        "noUnusedLocals": true,
         "target": "es2015",
         "noImplicitAny": false,
         "sourceMap": true,