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

Documentation rework & mu.semte.ch import #28

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Denperidge
Copy link

No description provided.

@abeforgit
Copy link

To be honest I don't really see the added value of the model creator section. The original article is eh, "sparse" at best, and doesn't really explain anything about mu-cl-resources.
There's some screenshots without code samples and the promise of an eventually functional tool, which doesn't exist yet...

Adding the link to mu-project definitely makes sense though

About json over js, it's true that it's more correct, but json is actually extremely annoying for documentation since it doesn't allow comments, and as you can see in the screenshot, things like ellipses get marked as red, where they're clearly just there for brevity:
image

It's a toss up between correctness and flexibility

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.

2 participants