Closed agilgur5 closed 2 years ago
This is gonna merge conflict with #404 -- please merge that first as it's a complicated one that a bunch of other code depends on ~EDIT: nvm, see below~ EDIT2: more of a "silent" merge conflict due to variable renaming, so will build this on top of #404 after all and fix the renaming issue. Sorry for any notification churn!
~NOTE: this is built on top of #404 as it uses the
context
changes made there. As such, I've marked this PR as "Draft" until #404 is merged.~ Rebased on top and marked as ready for review.Summary
Combine the two
tsconfig
files into one as the split is not currently necessary and adds unnecessary complexityDetails
checkTsConfig
is literally only used byparseTsConfig
merge the unit tests as well
parse-tsconfig.spec
, so only add the error casecheck-tsconfig
was longer in the past and more files were being created then for separation, so this may have made more sense then, but now it is unnecessary