-
Notifications
You must be signed in to change notification settings - Fork 227
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
Dev deps fix + cleanup + refactor #159
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome, thanks for putting this work together 🙌
This is good to be merged from my end.
btw is the semantic release still working?
Hmm, doesn't appear to be since the last merge, I think I omitted the semantic-release format on accident 😨 This one should trigger it Edit: @divyenduz yes, it's not working. There's an invalid NPM token apparently: #158 |
0da0a45
to
8619389
Compare
Removes some compilation issues during development.
8619389
to
4cbe5e5
Compare
4cbe5e5
to
9286d50
Compare
I will patch it up 👍 Edit: we have a new token now ✅ |
🎉 This PR is included in version 1.1.8 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sorry for the delay in getting around to some of these fixes, it's been hectic at work lately so I hope this PR makes up for it 🙂
typescript
to the installation instructions in the readme (Peer dependency not installed #151)For those that want to review this (hey @divyenduz 😛), the commits should be relatively atomic making it easier if you go through one commit at a time.