-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add tsdocs for everything and improve typings for readability #270
Conversation
Visit the preview URL for this PR (updated for commit 204fe48): https://gcode-preview--pr270-feature-document-par-j7zk964i.web.app (expires Wed, 12 Feb 2025 22:54:51 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 59bd114ae4847b32c2bba0b68620b9069a3e3531 |
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 know this is already merged, I was really curious about what docs were generated on your first pass!
Something small drew my attention, here's some feedback for a next iteration.
/** | ||
* Creates a new BuildVolume instance | ||
* @param x - Width in mm | ||
* @param y - Depth in mm | ||
* @param z - Height in mm | ||
* @param color - Color for visualization (default: 0x888888) | ||
*/ |
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.
There is also an inch mode (that does not impact anything really, except some arc support calculations). It is true that it's mm
by default, but not necessarily.
This PR adds a lot of docs to the parser. And it reworks some types for readability. Workings remain the exactly the same.
Added tsdocs for everything. Typedoc now generates more useful documentation.