-
Notifications
You must be signed in to change notification settings - Fork 21
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
Convert code base to Typescript #116
Conversation
I'm curious why you chose to remove |
Wow! 👏 🎉 I'm realizing I won't be able to review a diff this big in my spare time, especially with renaming the files and changes not being picked up Sorry. That's as far as I'll get. 😶🌫️ |
@seanpoulter, Thanks for taking a look. I know this is a tough PR to review, sorry. I gave up the piecemeal approach as it made the conversion much harder 😰... You already helped, thank you.🙏 It's nice to know that I am not alone on this! I figure we will bite the bullet and do this in one PR, but we won't release it to prod (thus the 32.0.0-beta.0 version), so we can have more time to test and fix when needed. Does that sound alright? |
Mainly to simplify the env and reduce maintenance overhead. Given we already have lint/CI to detect errors and prettier-write to correct them, I thought husky/lint-staged didn't really add any extra value... Besides, we didn't use them in |
Changes
tsc
.babel-jest
withts-test
to run Jest with better type error detection.index.d.ts
from the ts files.Backward compatibility
This PR does not change functionality, and I try to keep the types as backward-compatible as possible. However, there are still some breaking changes (mainly types). Therefore, mark this as "32.0.0-beta.0" to start preparing for the next major release.
Integration Testing
I have done some preliminary tests with
vscode-jest
and it seems to work fine. Would encourage other dependent systems to try out and let me know if there are any other unexpected issues.