-
Notifications
You must be signed in to change notification settings - Fork 35
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
Rearrange documentation and add "learn the workflow" section #223
Conversation
|
I would put it under recipes, since it is not really learning the workflow, it is some kind of feature you want to talk about. The Minilogo tutorial is also some kind of learning + multiple recipes, and I think maybe we should adapt it to fit more with the workflow. On the other hand... each project has its own challenges and therefore differs from the core workflow. |
I got some ideas in detail:
Regarding "Our Workflow":
For the "Workflow diagram":
Nevertheless, I like your clear steps a lot and I think, they will help newcomers a lot! |
Thank you, @JohannesMeierSE . I made all these changes. |
Add a hint on names for resolving scope. The name is the key to resolve cross-references. |
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 really like the new flow of the documentation! Thanks a lot @Lotes.
Just a couple minor suggestions
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.
Thank you very much for your PR, @Lotes. I like the new flow very much and also the additions you made. I just added a few detail remarks.
I reorganized the documentation in this PR. I do not want to delete any existing pages, but sometimes they are a bit long and misleading. Most of them are fine for me.
I also added a learning section where (new) the basic workflow with Langium gets explained.