Merge pull request #97 from Loathing-Associates-Scripting-Society/fix…

…-eslint-build-issues

Fix eslint build issues

.github/workflows/main.yml

@@ -14,11 +14,9 @@ jobs:
       - name: Run client ESLint & Prettier
         run: cd client && yarn run lint
       - name: Check client types
-        run: cd client && yarn run tsc
+        run: cd client && yarn run generate && yarn run tsc
       - name: Install server modules
         run: cd server && yarn install --frozen-lockfile
-      - name: Run server ESLint & Prettier
-        run: cd server && yarn run lint
       - name: Check server types
         run: cd server && yarn run build:types
 

client/eslint.config.js

@@ -9,6 +9,16 @@ import tseslint from "typescript-eslint";
 export default [
   eslint.configs.recommended,
   ...tseslint.configs.recommended,
+  {
+    ignores: [
+      "node_modules/**/*",
+      "build/**/*",
+      "src/generated/**/*",
+      "KoLmafia/**/*",
+      "config-overrides/**/*",
+      "src/setupProxy.js/**/*",
+    ],
+  },
   {
     languageOptions: {
       parserOptions: {
@@ -22,7 +32,6 @@ export default [
       "react-hooks": fixupPluginRules(reactHooksPlugin),
       libram: libramPlugin,
     },
-    ignores: ["/node_modules/**/*", "/build/**/*", "/src/generated/**/*"],
     rules: {
       "block-scoped-var": "error",
       curly: ["error", "multi-line"],

client/src/contexts/NagContext.tsx

@@ -16,7 +16,6 @@ export interface NagContextValue {
 
 const NagContext = createContext<NagContextValue>({
   nags: {},
-  // eslint-disable-next-line @typescript-eslint/no-empty-function
   withNag() {},
 });
 

client/src/contexts/RefreshContext.tsx

@@ -13,7 +13,6 @@ import React, { createContext, ReactNode, useEffect, useState } from "react";
 import useInterval from "../hooks/useInterval";
 import { markRemoteCallCacheDirty } from "../kolmafia/remote";
 
-// eslint-disable-next-line @typescript-eslint/no-empty-function
 export let triggerSoftRefresh = () => {};
 
 const RefreshContext = createContext({