Add eslint rules against promise misuse

pull/256/head
Candid Dauth 2024-03-02 02:55:43 +01:00
rodzic 9550c91dae
commit cbf51d148a
5 zmienionych plików z 15 dodań i 3 usunięć

Wyświetl plik

@ -3,13 +3,21 @@ module.exports = {
root: true,
ignorePatterns: ["**/dist/*", "**/out/*", "**/out.*/*"],
parserOptions: {
parser: "@typescript-eslint/parser"
parser: "@typescript-eslint/parser",
project: ["./*/tsconfig.json", "./*/tsconfig.node.json"],
extraFileExtensions: [".vue"]
},
plugins: ["@typescript-eslint", "import"],
extends: [
"plugin:import/typescript",
"plugin:vue/vue3-essential"
],
overrides: [
{
extends: ["plugin:@typescript-eslint/disable-type-checked"],
files: ["**/*.js", "**/*.cjs"]
}
],
env: {
node: true
},
@ -29,6 +37,8 @@ module.exports = {
"no-restricted-globals": ["error", "$"],
"no-restricted-imports": ["error", "vue/types/umd"],
"vue/multi-word-component-names": ["off"],
"@typescript-eslint/no-base-to-string": ["error"],
"@typescript-eslint/no-misused-promises": ["error", { checksVoidReturn: false }],
"constructor-super": ["error"],
"for-direction": ["error"],

Wyświetl plik

@ -10,7 +10,7 @@
let allItemsP: Promise<Record<string, string>>;
async function getAllItems(): Promise<Record<string, string>> {
if (!allItemsP) {
if (!allItemsP) { // eslint-disable-line @typescript-eslint/no-misused-promises
allItemsP = Promise.all(symbolList.map(async (s) => (
[s, await getSymbolHtml("currentColor", "1.5em", s)] as const
))).then((l) => Object.fromEntries(l));

Wyświetl plik

@ -6,6 +6,7 @@
},
"include": [
"vite.config.ts",
"vite-lib.config.ts",
"vite-define.ts",
"build.ts"
]

Wyświetl plik

@ -12,7 +12,7 @@ export function getFacilMapUrl(): string {
export async function openSocket<V extends SocketVersion>(version: V): Promise<Socket<SocketServerToClientEvents<V>, SocketClientToServerEvents<V>>> {
const serverUrl = new URL(getFacilMapUrl());
const socket = io(`${serverUrl.origin}${version !== SocketVersion.V1 ? `/${SocketVersion}` : ""}`, {
const socket = io(`${serverUrl.origin}${version !== SocketVersion.V1 ? `/${version}` : ""}`, {
forceNew: true,
path: serverUrl.pathname.replace(/\/$/, "") + "/socket.io"
});

Wyświetl plik

@ -196,6 +196,7 @@ export default class DatabaseHelpers {
}
const Pad = this._db.pads.PadModel.build({ id: padId } satisfies Partial<CreationAttributes<PadModel>> as any);
// eslint-disable-next-line @typescript-eslint/no-base-to-string
const objs: Array<Model> = await (Pad as any)["get" + this._db._conn.model(type).getTableName()](condition);
for (const obj of objs) {