-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat: type-genearation (only for exports) (fixes #98) #104
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.
Hello! 😄
Thanks for this PR, I am super excited for it!
So the code looks great, and everything is fine there! But, I'm a little confused on how to test this. Could you share your test project with me so I can test it myself?
Or even better, could we create a project in: https://github.com/torch2424/as-bind/tree/master/examples , like something similar to https://github.com/torch2424/as-bind/tree/master/examples/quickstart , so we can even test it in the CI? 😄
Let me know what you think! Thanks! 😄 🎉
@mathe42 this feature looks great, could you provide @torch2424 with info? |
Sorry the amount of time I can put into this is currently very small. but I will add a small example within the next 10 days. |
@torch2424 Added a small example. (That is not runnable but we get full types) |
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.
My apologies for the super late reply! I wasn't checking Github at all during the holidays so I could spend time with family and things. 😄
This is looking great to me! 🎉 If you don't mind, after I merge this I'll probably do a follow up PR, to move around the package.json entry, as well as probably edit the README a little bit.
If you are okay with that, I'll go ahead and merge this, and make that follow up PR 😄 Thank you SOOOOO much for all the hard work here! 🙏🏾 😄 🎉
This PR is the 3rd try to implement this properly. And basicly it now is a simple change + let TS handle the rest.
Imports I have not covered as #102 will change evrything. There a simple cli-script with onyly argument that points to the WASM file can be written.