-
-
Notifications
You must be signed in to change notification settings - Fork 15
fix: Added Typescript support #99
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: main
Are you sure you want to change the base?
Conversation
@dsanders11 @jviotti @robertgzr Could you please take a look at this PR? |
Co-authored-by: Jason Gwartz <jason@gwartz.me>
Thanks for the review. I've made the change. |
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.
I'm not a maintainer, just a fellow user - but I've tried out this change locally and it has worked for me, would be very useful to have merged! 🙏
@jviotti @robertgzr @dsanders11, Could you please take a look at this PR? It's been over 3 weeks since it was opened. |
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.
I don't have much cycles for Postject right now, but looks OK
It appears that the CI tests did not run. Could a maintainer please take a look? Thanks. |
Adds support for Typescript by adding a declaration for the API.
Fixes #98