Fix tsconfig, add build step in CI (#267)

trevor-scheer · web-flow · commit 01423741d607 · 2023-10-03T15:45:19.000-07:00
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -40,6 +40,7 @@ jobs:
     steps:
       - setup-node:
           node-version: <<parameters.node-version>>
+      - run: npm run build
       - run: npm run test:ci
       - store_test_results:
           path: junit.xml
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -4,6 +4,7 @@
 		"node": true,
 		"jest": true
 	},
+	"ignorePatterns": ["build", "node_modules"],
 	"parserOptions": {
 		"project": "./tsconfig.json"
 	},
diff --git a/tsconfig.cjs.json b/tsconfig.cjs.json
@@ -3,6 +3,7 @@
 	"compilerOptions": {
 		"target": "ES2021",
 		"module": "CommonJS",
+		"moduleResolution": "Node",
 		"outDir": "./build/cjs"
 	},
 	"include": ["src"]
diff --git a/tsconfig.esm.json b/tsconfig.esm.json
@@ -1,7 +1,6 @@
 {
 	"extends": ".",
 	"compilerOptions": {
-		"module": "ESNext",
 		"outDir": "./build/esm"
 	},
 	"include": ["src"]
diff --git a/tsconfig.json b/tsconfig.json
@@ -2,7 +2,8 @@
 	"extends": "@oly_op/tsconfig/tsconfig",
 	"compilerOptions": {
 		"lib": ["ES2021"],
-		"declaration": true
+		"declaration": true,
+		"module": "NodeNext"
 	},
 	"ts-node": {
 		"esm": true,
