-
Notifications
You must be signed in to change notification settings - Fork 521
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
refactor(examples): Examples overhaul #490
Conversation
… example, to cover potential issues
* moved a few of them to better folders * upgrading to es6 + V2 * particularly focusing on solidFromSlices/extrudeFromSlices
better structure, and more idiomatic v2
41ce1fe
to
d18a915
Compare
@kaosat-dev you seem to be pretty remote these days, so there will be additional contributions to this PR. if you happen to read this then can you commit the most recent changes? thanks |
Quick comment... web/package.json includes @jscad/examples as a 'dependency', which is incorrect. this package should be a 'devDependency'. |
@z3dev I think these are in pretty good shape now for alpha release. Please review, and give me an comments. All the basics are covered, but some of the more complex examples are still awaiting some attention. Those remain in the 'old' folder, and can come later. |
@SimonClark super job! I’ve been following along, and running examples. : ) I’ll do a more formal review of the code. It might take a day or two. |
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.
Some of the new examples are really cool. Thanks for taking this task on.
Some minor changes have been requested. I suggest running lint on the examples and cleaning up the code to meet standards.
As you make changes to close requests, please RESOLVE the request. I will review again once you are finished.
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.
Vastly improved examples. Thanks again.
I found a few things to change, then let’s merge for Alpha version.
Opened issue #676 to track a new task for the examples. Here’s a comparison of comments versus JSDOC tags. Really close. Comments (current):
Comments (JSDOC)
|
@SimonClark this looks like a good start for the examples. Any further changes coming? |
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.
@SimonClark this looks like a good start for the examples. Any further changes coming?
I got nothing planned imminently. Ship it!
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.
ALPHA version is done!!!
AWESOME!!!
This pr overhauls all the examples:
All Submissions: