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
Improve types #420
Improve types #420
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.
TS' type system is nice 👍
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.
Except for the default params @jeysal pointed out, this LGTM.
(enum vs string union - I'm -0 on changing since it's more verbose, but it's way easier to find usage, so I don't mind)
9188e59
to
b8c03ee
Compare
I'm going to merge this as I think the remaining unresolved comment isn't worth blocking on: if it is a problem, it'll mean rethinking our coverage, and/or waiting for a bug fix on another package. It's pretty easy to address in post if we want :) Cheers for the reviews! |
🎉 This PR is included in version 22.21.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Removes most of our type casts, making things a bit nicer.
This will cause a merge conflict w/ #419