Implement --skipLibCheck flag in tsbuild #45698
Closed
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.
Summary:
Many large composite projects rely on skipLibCheck to improve build times.
However, skipLibCheck does introduce some risk that projects miss typescript errors
in new node module versions they install.
This is in response to Issues #45691, #25613, and #41185
This issue is not tagged with Backlog, but I'd request that it be considered anyway. Reason being: is an issue with many duplicates and a long history of discussion. I don't believe that this change should be controversial.
[ ] There is an associated issue in the
Backlog
milestone (required)[x] Code is up-to-date with the
main
branch[x] You've successfully run
gulp runtests
locally[x] There are new or updated unit tests validating the change
Test Plan:
New tsbuild unit test scenario that tests overriding the skipLibCheck value with a command line argument.
Validated with verbose output that the actual build overrode the value in the tsconfig.json file.
Fixes #25613 (partially), #45691, and #41185