-
Notifications
You must be signed in to change notification settings - Fork 33
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
NPM Release and Typescript Support #13
Comments
Not yet. Perhaps NPM packaging sooner than Typescript support. I'll mark this as an enhancement. |
Are you open to contributions? If so, I'd be glad to try and help with these things when I've got some free time. |
Yes, this library is open to contributions! When you get the opportunity, feel free to fork, update, test, and create a pull request. |
@jwdunn1 Do you have any contact info I can reach you at? I want to run a few things by you before I begin really digging in. |
https://github.com/wuyudi/p5.EasyCam/blob/try2ts/dist/p5.easycam.d.ts I spent some time to adopt it into ts and generate a .d.ts file. Hope it may help. Some types are not clear to me, so I write I don't know whether this can be a PR (ts to js is a huge difference). |
@wuyudi Excellent! Thank you for the time spent on this. If you want to create a PR, please rename or drop the dist folder - it may be misunderstood as a distribution folder. |
NPM package published. |
The TypeScript version of the code is v1.2.0. This is not aligned with the current release version 1.2.2. |
Is there current Typescript Support and NPM packages for this library?
The text was updated successfully, but these errors were encountered: