-
Notifications
You must be signed in to change notification settings - Fork 10
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
Switch to Typescript #10
Switch to Typescript #10
Conversation
Oh wow, thanks for the contribution. This is a lot of work. It may take me a bit to review this but I'm absolutely open to making the switch to TS |
Great, nice to hear. Looking forward to your feedback. |
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.
Mostly looks good! We still have some .js
files I see, is that going to cause a problem? (I've never worked in a hybrid environment)
No, it should not cause any issues. But you don't have typing for the |
Any news on this? Waiting for me? |
Apologies. I caught COVID. I will try to take a look this weekend
…On Mon, Aug 14, 2023, 12:12 Ruedi Steinmann ***@***.***> wrote:
Any news on this? Waiting for me?
—
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAX6K6O57GNRZY264GQDHJDXVJFALANCNFSM6AAAAAA24VPEP4>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
My Anycubic Photon had a hard time with the generated .pws file, so I wanted to add support for .photon. I had a hard time understanding the code and thought some types would greatly help.
I also dropped the checked in libraries and switched to npm packages.
Oh, and I also changed the anti-alias count from 0 to 1. UVTools had a division by zero before...
What do you think?