From f499507698e0f940551e5f4ee0a82d82f609dcae Mon Sep 17 00:00:00 2001 From: dank074 Date: Wed, 13 Nov 2024 19:05:54 -0600 Subject: [PATCH] turn off eslint rules causing issues --- eslint.config.mjs | 2 ++ src/api/middlewares/Authentication.ts | 1 - src/api/routes/auth/reset.ts | 1 - src/gateway/opcodes/Identify.ts | 1 - src/gateway/opcodes/LazyRequest.ts | 1 - src/gateway/opcodes/VoiceStateUpdate.ts | 1 - src/util/entities/BaseClass.ts | 2 +- src/util/schemas/MessageCreateSchema.ts | 1 - src/util/schemas/responses/TeamListResponse.ts | 1 + src/util/util/AutoUpdate.ts | 1 - src/util/util/Event.ts | 1 + src/util/util/Sentry.ts | 2 +- 12 files changed, 6 insertions(+), 9 deletions(-) diff --git a/eslint.config.mjs b/eslint.config.mjs index a4209123..e321115c 100644 --- a/eslint.config.mjs +++ b/eslint.config.mjs @@ -41,5 +41,7 @@ export default [{ "no-mixed-spaces-and-tabs": "off", "@typescript-eslint/no-inferrable-types": "off", // Required by typeorm "@typescript-eslint/no-var-requires": "off", // Sometimes requred by typeorm to resolve circular deps + "@typescript-eslint/no-require-imports": "off", + "@typescript-eslint/no-unused-vars": "off", }, }]; \ No newline at end of file diff --git a/src/api/middlewares/Authentication.ts b/src/api/middlewares/Authentication.ts index b5204ffc..b4f48477 100644 --- a/src/api/middlewares/Authentication.ts +++ b/src/api/middlewares/Authentication.ts @@ -108,7 +108,6 @@ export async function Authentication( req.rights = new Rights(Number(user.rights)); return next(); } catch (error) { - // eslint-disable-next-line @typescript-eslint/no-non-null-assertion return next(new HTTPError(error!.toString(), 400)); } } diff --git a/src/api/routes/auth/reset.ts b/src/api/routes/auth/reset.ts index b3ca1e9e..da4d3334 100644 --- a/src/api/routes/auth/reset.ts +++ b/src/api/routes/auth/reset.ts @@ -72,7 +72,6 @@ router.post( await User.update({ id: user.id }, data); // come on, the user has to have an email to reset their password in the first place - // eslint-disable-next-line @typescript-eslint/no-non-null-assertion await Email.sendPasswordChanged(user, user.email!); res.json({ token: await generateToken(user.id) }); diff --git a/src/gateway/opcodes/Identify.ts b/src/gateway/opcodes/Identify.ts index 94320eee..c6f987fc 100644 --- a/src/gateway/opcodes/Identify.ts +++ b/src/gateway/opcodes/Identify.ts @@ -171,7 +171,6 @@ export async function onIdentify(this: WebSocket, data: Payload) { // but we do want almost everything from guild. // How do you do that without just enumerating the guild props? guild: Object.fromEntries( - // eslint-disable-next-line @typescript-eslint/no-non-null-assertion getDatabase()! .getMetadata(Guild) .columns.map((x) => [x.propertyName, true]), diff --git a/src/gateway/opcodes/LazyRequest.ts b/src/gateway/opcodes/LazyRequest.ts index 45eaccfa..eba284b2 100644 --- a/src/gateway/opcodes/LazyRequest.ts +++ b/src/gateway/opcodes/LazyRequest.ts @@ -214,7 +214,6 @@ async function subscribeToMemberEvents(this: WebSocket, user_id: string) { export async function onLazyRequest(this: WebSocket, { d }: Payload) { // TODO: check data check.call(this, LazyRequestSchema, d); - // eslint-disable-next-line @typescript-eslint/no-unused-vars const { guild_id, typing, channels, activities, members } = d as LazyRequestSchema; diff --git a/src/gateway/opcodes/VoiceStateUpdate.ts b/src/gateway/opcodes/VoiceStateUpdate.ts index ffc143ed..b45c8203 100644 --- a/src/gateway/opcodes/VoiceStateUpdate.ts +++ b/src/gateway/opcodes/VoiceStateUpdate.ts @@ -99,7 +99,6 @@ export async function onVoiceStateUpdate(this: WebSocket, data: Payload) { voiceState.token = genVoiceToken(); voiceState.session_id = this.session_id; - // eslint-disable-next-line @typescript-eslint/no-unused-vars const { id, ...newObj } = voiceState; await Promise.all([ diff --git a/src/util/entities/BaseClass.ts b/src/util/entities/BaseClass.ts index f4b3cf59..0171e010 100644 --- a/src/util/entities/BaseClass.ts +++ b/src/util/entities/BaseClass.ts @@ -46,7 +46,7 @@ export class BaseClassWithoutId extends BaseEntity { // eslint-disable-next-line @typescript-eslint/no-explicit-any toJSON(): any { return Object.fromEntries( - // eslint-disable-next-line @typescript-eslint/ban-ts-comment, @typescript-eslint/no-non-null-assertion + // eslint-disable-next-line @typescript-eslint/ban-ts-comment this.metadata!.columns // @ts-ignore .map((x) => [x.propertyName, this[x.propertyName]]) .concat( diff --git a/src/util/schemas/MessageCreateSchema.ts b/src/util/schemas/MessageCreateSchema.ts index 15537ca8..4eaf7c96 100644 --- a/src/util/schemas/MessageCreateSchema.ts +++ b/src/util/schemas/MessageCreateSchema.ts @@ -69,7 +69,6 @@ export interface MessageCreateSchema { } // TypeScript complains once this is used above -// eslint-disable-next-line @typescript-eslint/no-unused-vars interface PollCreationSchema { question: PollMedia; answers: PollAnswer[]; diff --git a/src/util/schemas/responses/TeamListResponse.ts b/src/util/schemas/responses/TeamListResponse.ts index 375f0ae6..c4a7d177 100644 --- a/src/util/schemas/responses/TeamListResponse.ts +++ b/src/util/schemas/responses/TeamListResponse.ts @@ -18,4 +18,5 @@ import { Team } from "@spacebar/util"; +// eslint-disable-next-line @typescript-eslint/no-empty-object-type export interface TeamListResponse extends Array {} diff --git a/src/util/util/AutoUpdate.ts b/src/util/util/AutoUpdate.ts index b7721cdb..9a3c0a2a 100644 --- a/src/util/util/AutoUpdate.ts +++ b/src/util/util/AutoUpdate.ts @@ -70,7 +70,6 @@ export function enableAutoUpdate(opts: { }); } -// eslint-disable-next-line @typescript-eslint/no-unused-vars async function download(url: string, dir: string) { try { // TODO: use file stream instead of buffer (to prevent crash because of high memory usage for big files) diff --git a/src/util/util/Event.ts b/src/util/util/Event.ts index 01f4911a..bbc93aac 100644 --- a/src/util/util/Event.ts +++ b/src/util/util/Event.ts @@ -100,6 +100,7 @@ export async function listenEvent( }; const listener = (msg: ProcessEvent) => { + // eslint-disable-next-line @typescript-eslint/no-unused-expressions msg.type === "event" && msg.id === event && callback({ ...msg.event, cancel }); diff --git a/src/util/util/Sentry.ts b/src/util/util/Sentry.ts index 510ed453..005b09e7 100644 --- a/src/util/util/Sentry.ts +++ b/src/util/util/Sentry.ts @@ -117,7 +117,7 @@ export const Sentry = { Integrations.setupExpressErrorHandler(app); // The typings for this are broken? - // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars + // eslint-disable-next-line @typescript-eslint/no-explicit-any app.use(function onError(err: any, req: any, res: any, next: any) { res.statusCode = 500; res.end(res.sentry + "\n");