Skip to content

Project TSConfig Checking#159

Open
Offroaders123 wants to merge 11 commits intojohnnesky:mainfrom
Offroaders123:tsconfig
Open

Project TSConfig Checking#159
Offroaders123 wants to merge 11 commits intojohnnesky:mainfrom
Offroaders123:tsconfig

Conversation

@Offroaders123
Copy link

This enables project-wide TSConfig type checking, and fixes various errors and warnings which arose after enabling it.

This is the command I set up to demo this as well.

`tsc --noEmit --skipLibCheck --target esnext --module nodenext ./**/*.ts`

I have a 'template' config that I tend to start with, then I remove flags to make it on the same level as what the project needs, then bring flags back into place one by one, incrementally allowing for stronger type checking. :)

So right now this simply allows you to run `tsc` on the project, and you get a list of all current errors and issues to focus on, without having to go to each file to find them. It helps for migrations and larger cross-project updates as well.
Enabled a few more flags, seemingly no errors for those (yay!). A few strict-related flags rose some errors, so those have been omitted for this commit, so they can be resolved each on their own.
Errors present! Will resolve next commit.
Allows for compilation without needing to use ts, having incremental compilation (I think that's the name?) works when you mark types as types when exported from a module.
Some of the more strict TSConfig flags would require intent oversight, as it would change concepts of how the code functions, which I don't want to mangle with specifically. And I don't want to introduce a whole large list of sweeping changes either.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant