Skip to content
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

Switch entire codebase to TypeScript #3110

Merged
merged 5 commits into from
May 25, 2021
Merged

Switch entire codebase to TypeScript #3110

merged 5 commits into from
May 25, 2021

Conversation

IvanGoncharov
Copy link
Member

@saihaj Did an excellent job in #2828 and maintained it for months.
Now is time to merge it into main, the goal is to use code freeze (see #3087) as some of the opportunity to replace workaround with correct fixes.
Deadline: Friday, May 21 23:59 UTC

@IvanGoncharov IvanGoncharov added the PR: breaking change 💥 implementation requires increase of "major" version number label May 18, 2021
@IvanGoncharov IvanGoncharov force-pushed the ts_main branch 29 times, most recently from baa6a5d to 9271b32 Compare May 24, 2021 23:55
@IvanGoncharov IvanGoncharov force-pushed the ts_main branch 2 times, most recently from c18ce9a to 218a6b4 Compare May 25, 2021 00:41
@IvanGoncharov IvanGoncharov merged commit c589c3d into main May 25, 2021
@IvanGoncharov IvanGoncharov deleted the ts_main branch May 25, 2021 00:56
@IvanGoncharov IvanGoncharov changed the title Switch to entire codebase to TypeScript Switch entire codebase to TypeScript May 25, 2021
@mike-marcacci
Copy link
Contributor

mike-marcacci commented May 25, 2021

W00T!! Congrats @saihaj and @IvanGoncharov on landing this epic effort!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: breaking change 💥 implementation requires increase of "major" version number
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants