Skip to content

Commit

Permalink
Fix some linter issues
Browse files Browse the repository at this point in the history
  • Loading branch information
thaapasa committed Sep 28, 2024
1 parent 7fc2841 commit de54179
Show file tree
Hide file tree
Showing 6 changed files with 28 additions and 16 deletions.
12 changes: 12 additions & 0 deletions eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,18 @@ export default tseslint.config({
'unused-imports/no-unused-imports': 'error',
'simple-import-sort/imports': ['warn'],
'simple-import-sort/exports': ['warn'],
"@typescript-eslint/no-explicit-any": "off",
"@typescript-eslint/no-empty-object-type": "off",
"@typescript-eslint/no-unused-vars": [
"warn",
{
"argsIgnorePattern": "^_",
"caughtErrorsIgnorePattern": "^_",
"destructuredArrayIgnorePattern": "^_",
"varsIgnorePattern": "^_",
"ignoreRestSiblings": true
}
]
}
})

Expand Down
2 changes: 1 addition & 1 deletion src/app/calc/checks.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ export const checkLuhn = (str: string) => {
try {
const fullValue = luhn.generate(str);
return fullValue && fullValue.length > str.length ? fullValue[fullValue.length - 1] : '-';
} catch (e) {
} catch (_e: any) {
return '-';
}
};
4 changes: 2 additions & 2 deletions src/app/calc/xml-json.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export function xmlToJson<T>(x: string): Promise<T> {
export async function xmlToJsonString(x: string): Promise<string> {
try {
return JSON.stringify(await xmlToJson(x), null, 2);
} catch (e) {
} catch (_e) {
return 'Invalid XML';
}
}
Expand All @@ -39,7 +39,7 @@ export function jsonToXml(x: any): string {
export function jsonStringToXml(x: string): string {
try {
return jsonToXml(JSON.parse(x));
} catch (e) {
} catch (_e) {
return 'Invalid JSON';
}
}
6 changes: 3 additions & 3 deletions src/app/ui/TextConversionPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -23,23 +23,23 @@ interface ConverterInfo {
function toPrettyJSON(s: string): string {
try {
return JSON.stringify(JSON.parse(s), null, 2);
} catch (e) {
} catch (_e) {
return s;
}
}

function toCompactJSON(s: string): string {
try {
return JSON.stringify(JSON.parse(s));
} catch (e) {
} catch (_e) {
return s;
}
}

function svgToRn(s: string): string {
try {
return svgToReactNative(s);
} catch (e) {
} catch (_e) {
return 'Virheellinen SVG';
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/app/util/store.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export function getValue<T>(key: string): T | null {
try {
const v = window.localStorage.getItem(key);
return v !== null ? (JSON.parse(v) as T) : null;
} catch (e) {
} catch (_e) {
return null;
}
} else {
Expand Down
18 changes: 9 additions & 9 deletions src/app/util/util.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,23 +10,23 @@ export function getRandomInt(min: number, max: number): number {
return Math.floor(Math.random() * (max - min)) + min;
}

export function isDefined(value: any): boolean {
export function isDefined(value: unknown): boolean {
return value !== undefined && value !== null;
}

export function isNumber(value: any): boolean {
export function isNumber(value: unknown): boolean {
return value !== undefined && value !== null && typeof value === 'number' && !isNaN(value);
}

export function isString(value: any): boolean {
export function isString(value: unknown): boolean {
return value !== undefined && value !== null && typeof value === 'string';
}

export function isObject(value: any): boolean {
export function isObject(value: unknown): boolean {
return value !== undefined && value !== null && typeof value === 'object';
}

export function isArray(value: any): boolean {
export function isArray(value: unknown): boolean {
return Array.isArray(value);
}

Expand All @@ -48,12 +48,12 @@ export function nonEmpty(value: any): boolean {
return value && value.length && value.length > 0;
}

export function combine(a: any, b: any): string {
export function combine(a: unknown, b: unknown): string {
return `${a}${b}`;
}

export function combineWith(separator: string): (a: any, b: any) => string {
return (a: any, b: any) => `${a}${separator}${b}`;
export function combineWith(separator: string): (a: unknown, b: unknown) => string {
return (a: unknown, b: unknown) => `${a}${separator}${b}`;
}

export function pairsToObject<T>(pairs: Array<[string, T]>): Record<string, T> {
Expand All @@ -63,7 +63,7 @@ export function pairsToObject<T>(pairs: Array<[string, T]>): Record<string, T> {
}

export function objectKeys<T extends object>(object: T): ReadonlyArray<keyof T> {
return Object.keys(object) as any;
return Object.keys(object) as unknown as ReadonlyArray<keyof T>;
}

export function mapObject<S, T, O, X extends { [k in keyof O]: S }>(
Expand Down

0 comments on commit de54179

Please sign in to comment.