-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
☂️ Fix remaining TS/TSX static type errors #5993
Copy link
Copy link
Open
6 / 66 of 6 issues completedLabels
epicteam-mobile-platformMobile Platform teamMobile Platform teamtype-tech-debtTechnical debtTechnical debt
Description
Describe the issue
Increase ability to catch bugs at compile time. I.e. identify and resolve faster.
This is a follow-up to
which enables TSC to emit our remaining TS/TSX type errors to fix
~570ish errors in ~90ish files, tracked in our internal google sheet
once complete at 0 errors (and 0 warnings ideally)
we can re-enable TSC in our pre-commit hooks and CI
- TS check isn't working #4605
- already partially complete in ci: Add type check lint to CI #7100
then later upgrade to TypeScript v5 once our MM shared ESLint config is on v5
a good long-term follow-up would be
- converting remaining
.jsfiles totsremains, a huge task worthy of it's own umbrella issue / epic- e.g. app/util/transactions/index.js -> app/util/transactions/index.ts (no changes) exposes
- Found 234 errors in 48 files -> Found 394 errors in 50 files
Screenshots
In issues above ^
To Reproduce
yarn tsc
Expected behavior
0 errors
0 warnings (ideally)
Smartphone (please complete the following information):
N/A
to be added after bug submission by internal support / PM
Severity
- How critical is the impact of this bug on a user?
- Add stats if available on % of customers impacted
- Is this visible to all users?
- Is this tech debt? yes
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
epicteam-mobile-platformMobile Platform teamMobile Platform teamtype-tech-debtTechnical debtTechnical debt