fix: Separated type checks from extension building [#1441]
This MR contains the following changes:
- Removed
--type-checking
build argument in favor of runningtsc --noEmit
.- Merged
npm run lint:ts-only
andnpm run test:types
intonpm run $ lint.ts
, which is executed as part ofnpm run lint
. - Updated documentation accordingly.
- Merged
- Updated TypeScript configuration to only check files in the src/ directory.
Note that this means that TypeScript files in the js/api/ directory aren't checked directly, but only indirectly via imports. We could also includejs/**/*.ts
in the configuration, if we want to make this more explicit, but this would become unnecessary as soon as we move those files over to the src/ directory.
This MR also contains the following unrelated changes:
- Cleaned up TypeScript-specific npm scripts.
- Removed redundant
npm run prettier:ts-only
in favor ofnpm run prettier
. - Merged
npm run test:unit:printCoverageLink
intonpm run $ test.unit
, which is executed as part ofnpm test
. - Merged
npm run test:unit:ts-only
intonpm run $ test.unit
, which is executed as part ofnpm test
.
- Removed redundant
- Cleaned up TypeScript configuration.
- Rearranged properties alphabetically to make it easier to read.
- Removed unused "outDir" property, since we're not directly using
tsc
for generating JavaScript files, but only via webpack.