-
Notifications
You must be signed in to change notification settings - Fork 20
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
Typescript: Enable strict mode #728
Conversation
hm, i see 94 errors when linking main and 94 errors when linking this branch. Are you sure this brings any improvement? |
same result here 94 errors before and after. Strict mode can be annoying with extra errors but no additional errors here so would be for enabling strict mode |
@dprokop well it fixes a few strict errors (see the ones fixed in PR) so it does something, not sure what else to enable |
@dprokop is there any argument to not extend tsconfig (Which has the important strict: true option) |
@Sergej-Vlasov not sure what your mean :) |
Sorry, meant to say that I support enabling strict mode in short 😅 |
🚀 PR was released in |
Should fix some of the errors that show up when yarn linking into main repo