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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add typescript #71
Add typescript #71
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.
This looks great! Thanks for adding it. Just a few comments to take a look at.
The indentation issue was only about my formatter 馃槄 |
Excellent, thank you again! This is a great addition. If you're still interested in digging into the code a bit more adding some documentation for new classes #72 would be another good place to dive in - I can help provide insight on what some of these different classes do and why, as well. Of course no pressure, though, I appreciate the help! |
Related issue: #65
For example, the Constants type, the HalfEdgeHelper class, I added the code but it is commented because I'm not sure which is the correct type of the halfEdge parameter.
Maybe it's all a mess 馃槄, please tell me what you think.