Skip to content

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 SimenB merged commit d0a8428 into master Jul 22, 2019
@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.

4 participants