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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: convert to TS #305

Merged
merged 21 commits into from Jul 22, 2019
Merged

chore: convert to TS #305

merged 21 commits into from Jul 22, 2019

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Jul 17, 2019

Take 2 of the TS migration.

Needs to wait for typescript-eslint/typescript-eslint#712 to avoid peer dep warning when installing this, but I think this now works beyond that 馃檪

~^ is released as canary. We should either wait for a stable or their next major, then we can merge this 馃 ~

^ Out as 1.13, will merge this

/cc @jeysal

@SimenB SimenB mentioned this pull request Jul 17, 2019
35 tasks
@SimenB
Copy link
Member Author

SimenB commented Jul 20, 2019

@G-Rath I'll rebase this after merging #255 - you might need to rebase your PRs as well. I'm happy to do it, I just don't wanna step on your toes if you're currently working on any of them 馃檪

@G-Rath
Copy link
Collaborator

G-Rath commented Jul 20, 2019

@SimenB you're welcome to rebase them - I suspect you're better at it anyway 馃槈

Right now I'm busy trying to figure out how to make a FunctionDeclaration AST node have a null body for coverage testing 馃槵

@SimenB
Copy link
Member Author

SimenB commented Jul 20, 2019

Cool, I'll rebase all of your open PRs, then 馃檪 I'll ping you if tests fail in any of them and they're not trivial to fix

@SimenB SimenB force-pushed the reapply-ts branch 3 times, most recently from 3a2687e to 68c2d89 Compare July 21, 2019 13:06
@SimenB
Copy link
Member Author

SimenB commented Jul 22, 2019

馃帀 This PR is included in version 22.13.1 馃帀

The release is available on:

Your semantic-release bot 馃摝馃殌

@SimenB SimenB deleted the reapply-ts branch May 9, 2020 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants