Skip to content
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

Merged
merged 42 commits into from
Sep 7, 2020
Merged

refactor(examples): Examples overhaul #490

merged 42 commits into from
Sep 7, 2020

Conversation

kaosat-dev
Copy link
Contributor

@kaosat-dev kaosat-dev commented Dec 16, 2019

This pr overhauls all the examples:

  • V2 API
  • more clear
  • more specific examples
  • docstrings & explanations
  • etc

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Does your submission pass tests?

@z3dev
Copy link
Member

z3dev commented Aug 4, 2020

@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

@z3dev
Copy link
Member

z3dev commented Aug 26, 2020

Quick comment...

web/package.json includes @jscad/examples as a 'dependency', which is incorrect. this package should be a 'devDependency'.

@SimonClark
Copy link
Contributor

@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.

@z3dev
Copy link
Member

z3dev commented Sep 1, 2020

@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.

Copy link
Member

@z3dev z3dev left a 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.

packages/examples/core/colors/colorCube.js Show resolved Hide resolved
packages/examples/core/colors/transparency.js Outdated Show resolved Hide resolved
packages/examples/core/colors/transparency.js Show resolved Hide resolved
packages/examples/parameters/gear.js Outdated Show resolved Hide resolved
packages/examples/parameters/gear.js Outdated Show resolved Hide resolved
packages/web/src/ui/views/examples.js Show resolved Hide resolved
Copy link
Member

@z3dev z3dev left a 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.

@z3dev
Copy link
Member

z3dev commented Sep 7, 2020

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)

  • @author : Identify the author(s) of an item.
  • @copyright : Document some copyright information.
  • @description : A full general description.
  • @file : Describe a file. (Use at start of file)
  • @license : Identify the license that applies to this code.
  • @see : Refer to some other documentation for more information.
  • @summary : A shorter version of the full description. (For doclets)
  • {@link} : Link to another item in the documentation.

@z3dev
Copy link
Member

z3dev commented Sep 7, 2020

@SimonClark this looks like a good start for the examples. Any further changes coming?

Copy link
Contributor

@SimonClark SimonClark left a 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!

packages/examples/core/hulls/hull.js Outdated Show resolved Hide resolved
packages/examples/parameters/allParamTypes.js Outdated Show resolved Hide resolved
Copy link
Member

@z3dev z3dev left a 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!!!

@z3dev z3dev merged commit d73e06f into V2 Sep 7, 2020
@z3dev z3dev deleted the examples-overhaul branch September 25, 2020 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants