FUSETOOLS2-2304: Change code to match new rules

.eslintrc.json

@@ -33,14 +33,21 @@
 			"error",
 			{
 			  "selector": "variable",
-			  "format": ["camelCase", "UPPER_CASE"]
+			  "format": ["camelCase", "UPPER_CASE"],
+			  "leadingUnderscore": "allow"
 			}
 		  ],
 		"@stylistic/semi": "warn",
 		"curly": "warn",
 		"eqeqeq": "warn",
-		"@typescript-eslint/no-unused-vars": "warn",
-		"no-throw-literal": "warn",
+		"@typescript-eslint/no-unused-vars":[
+			"warn",
+			{
+			  "argsIgnorePattern": "^_",
+			  "varsIgnorePattern": "^_",
+			  "caughtErrorsIgnorePattern": "^_"
+			}
+		  ],
 		"@typescript-eslint/no-floating-promises": "warn",
 		"import/no-extraneous-dependencies": [
 			"error",

src/conditions/Repeat.ts

@@ -88,7 +88,7 @@ class RepeatManager {
 	}
 
 	abortAll(): void {
-		for (const [index, repeat] of this._repeats.entries()) {
+		for (const [_index, repeat] of this._repeats.entries()) {
 			repeat.abort(undefined);
 			this.remove(repeat);
 		}
@@ -342,8 +342,7 @@ export class Repeat<T> {
 		}
 		else if (this._usingExplicitLoopSignaling === false || this._finishedLoop) {
 			if (this.options?.ignoreLoopError) {
-				// @ts-expect-error
-				// output will be ignored
+				// @ts-expect-error: output will be ignored
 				this.resolve(undefined);
 			}
 			else {

src/conditions/TimeoutPromise.ts

@@ -202,8 +202,7 @@ class TimeoutPromise<T> extends Promise<T> {
      */
 	static createFrom<T>(promise: Promise<T>, timeout?: number, options?: TimeoutPromiseOptions): TimeoutPromise<T> {
 		return new TimeoutPromise<T>((resolve, reject) => {
-			promise.then(resolve);
-			promise.catch(reject);
+			promise.then(resolve).catch(reject);
 		}, timeout, options);
 	}
 }

src/utils/Errors.ts

@@ -1,5 +1,6 @@
 import { error } from "vscode-extension-tester";
 
+// eslint-disable-next-line @typescript-eslint/ban-types
 export type ErrorType = Function;
 
 /**