Description
Nothing in CI typechecks test files, so type errors in *.test.ts and *.test.tsx can land on main unnoticed.
tsconfig.build.json excludes **/*.test.{js,jsx,ts,tsx} from the build, and npm run test transpiles through Babel, which does no typechecking. The error fixed in #288 was only caught locally by npm run dev, because dev:shell runs fork-ts-checker-webpack-plugin against the root tsconfig.json, whose include does cover test files.
Adding a tsc --noEmit -p tsconfig.json step to .github/workflows/ci.yml would close the gap.
Description
Nothing in CI typechecks test files, so type errors in
*.test.tsand*.test.tsxcan land on main unnoticed.tsconfig.build.jsonexcludes**/*.test.{js,jsx,ts,tsx}from the build, andnpm run testtranspiles through Babel, which does no typechecking. The error fixed in #288 was only caught locally bynpm run dev, becausedev:shellrunsfork-ts-checker-webpack-pluginagainst the roottsconfig.json, whoseincludedoes cover test files.Adding a
tsc --noEmit -p tsconfig.jsonstep to.github/workflows/ci.ymlwould close the gap.