Disable ts-loader typechecking for non-build files (resolves #208) #210
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Up until now the ts loader has been running in the mode where it runs all typechecking (like what you get in the VScode console for all the types in the project). Because this feature doesn't naturally support webpack caching, disabling it is the easiest fix for #208.
I tend to think a more permissive build makes sense as well and I believe is in line with what Deno offers too.
The main concern with an alternative fix like @sokra describes is that it would mean always loading the ts loader upfront (therefore slowing down startup) unless we can get a hook into the cache usage to know if a ".ts" file is being loaded or not in order to conditionally do this (perhaps Tobias will know if this is possible).
But it seems easier and a better experience I think to just go for the one-liner :)