Ditch ts-node - I'm pretty sure sku lint would typecheck it?

seek-oss · Dec 16, 2024 · 206679b · 206679b
1 parent d7297c6
commit 206679b
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 23 deletions.
diff --git a/fe/package.json b/fe/package.json
@@ -39,7 +39,6 @@
     "react-router-dom": "6.22.3",
     "scoobie": "18.1.1",
     "storybook": "8.4.7",
-    "ts-node": "10.9.2",
     "use-query-params": "2.2.1",
     "webpack-merge": "6.0.1"
   },
@@ -68,8 +67,7 @@
     "build": "sku build",
     "build-storybook": "storybook build --output-dir dist-storybook",
     "format": "sku format",
-    "lint": "sku lint && pnpm -s lint:codegen",
-    "lint:codegen": "ts-node lib/types/seekApi.typeDefs.ts",
+    "lint": "sku lint",
     "serve": "sku serve",
     "start": "sku start",
     "start-storybook": "storybook dev",

diff --git a/fe/sku.config.ts b/fe/sku.config.ts
@@ -71,13 +71,6 @@ const config: SkuConfig = {
       // under `browser` in package.json
       customConditions: ['browser'],
     },
-    // Override the compiler options when running ts-node
-    // to handling linting of typedefs
-    'ts-node': {
-      compilerOptions: {
-        module: 'nodenext',
-      },
-    },
     include: ['**/*', '.storybook/*'],
   }),
 };

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml