refactor(site): match v1 tsconfig; simplify tsconfig.test.json (#426)

* refactor(site): match v1 tsconfig

Summary of changes:

* ordered configurations for ease of readability
* allowJs removed -> not needed after Next.js
* noImplicitAny removed -> this is set to true by strict
* strictNullChecks removed -> this is set to true by strict
* removed lingering next-env.d.ts in include

* refactor(site): simplify tsconfig.test.json

Because that the base tsconfig was simplified when changing from
Next.js, this one now duplicates most of it. I still believe having a
separate tsconfig is a good idea for the purposes of separating tests
from source code. It especially can make it easier for eslint and jest
performance.
This commit is contained in:
G r e y
2022-03-13 19:54:35 -04:00
committed by GitHub
parent ec077c6191
commit 6c83907b1b
2 changed files with 11 additions and 23 deletions
+11 -14
View File
@@ -1,23 +1,20 @@
{
"compilerOptions": {
"outDir": "./dist/",
"noImplicitAny": true,
"module": "commonjs",
"target": "es5",
"jsx": "react",
"allowJs": true,
"downlevelIteration": true,
"esModuleInterop": true,
"resolveJsonModule": true,
"isolatedModules": true,
"lib": ["dom", "dom.iterable", "esnext"],
"skipLibCheck": true,
"strict": true,
"strictNullChecks": true,
"forceConsistentCasingInFileNames": true,
"incremental": true,
"moduleResolution": "node"
"isolatedModules": true,
"jsx": "react",
"lib": ["dom", "dom.iterable", "esnext"],
"module": "commonjs",
"moduleResolution": "node",
"outDir": "./dist/",
"resolveJsonModule": true,
"skipLibCheck": true,
"strict": true,
"target": "es5"
},
"include": ["next-env.d.ts", "**/*.ts", "**/*.tsx"],
"include": ["**/*.ts", "**/*.tsx"],
"exclude": ["node_modules", "_jest", "**/*.test.tsx"]
}
-9
View File
@@ -1,13 +1,4 @@
{
"extends": "./tsconfig.json",
"compilerOptions": {
"target": "es5",
"module": "commonjs",
"jsx": "react",
"downlevelIteration": true,
"strict": true,
"strictNullChecks": true,
"esModuleInterop": true
},
"exclude": ["node_modules", "_jest"]
}