-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update tsconfig for ts 4.0 #465
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
tsconfig.json
Outdated
"jest/*", | ||
"**/*.svg" | ||
] | ||
"include": ["src/**/*", "App.tsx", "**/*.d.ts"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
curious why the root folder is not included?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose because src/ and App.tsx are the only ts files? but no reason to not have it be the root, we will update!
8b57423
to
5bd659b
Compare
tsconfig.json
Outdated
"jest/*", | ||
"**/*.svg" | ||
] | ||
"include": ["**/*.ts", "**/*.d.ts"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we have the tsx files here too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe it works for tsx
if we use ts
but it might be good to add tsx
to be exhaustive?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are captured!
5bd659b
to
ae82aa1
Compare
Why: Currently our tsconfig is set to only run the ts compiler on files that end with `.d.ts` which means that the complier is not running on any of our source files. This was caused by an update to the behaviour of the includes tsconfig option from being inclusive to exclusive. This commit: updates the tsconfig file to fix the issue as well as removes no no longer necessary config options. We updated some type errors that made their way into the codebase during the few days of not type checking the entire codebase Co-Authored-By: devin jameson <devin@thoughtbot.com>
ae82aa1
to
451cccd
Compare
Why: Currently our tsconfig is set to only run the ts compiler on files that end with `.d.ts` which means that the complier is not running on any of our source files. This was caused by an update to the behaviour of the includes tsconfig option from being inclusive to exclusive. This commit: updates the tsconfig file to fix the issue as well as removes no no longer necessary config options. We updated some type errors that made their way into the codebase during the few days of not type checking the entire codebase Co-Authored-By: devin jameson <devin@thoughtbot.com> Co-authored-by: devin jameson <devin@thoughtbot.com>
Why:
Currently our tsconfig is set to only run the ts compiler on files that
end with
.d.ts
which means that the complier is not running on any ofour source files. This was caused by an update to the behaviour of the
includes tsconfig option from being inclusive to exclusive.
This commit:
updates the tsconfig file to fix the issue as well as removes no no
longer necessary config options. We updated some type errors that made
their way into the codebase during the few days of not type checking the
entire codebase
Co-Authored-By: devin jameson devin@thoughtbot.com