-
Notifications
You must be signed in to change notification settings - Fork 15
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
📝(project) add base documentation #60
Conversation
95aaf52
to
c88f8a0
Compare
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.
Beautiful Documentation! Thanks a lot Julien)
Thank you for your feedback @SergioSim ❤️ |
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.
Congrats for this comprehensive work!
There is still a very very small issue that comes from the automated update for command
and model
documentations:
- sometimes, there is the description sentence with a point (for example in Open edx events > base > baseContextModel)
Represents the base context model inherited by all event contexts.
- and sometimes not (for example in ralph > extract)
Extract input events from a container format using a dedicated parser
You are totally right, we need to stay consistent. IMO this should be addressed in a new PR. |
@SergioSim & @quitterie-lcs I've addressed all your comments in separated fixup commits. Is there any chance that you have time for a last review (and validation)? 🙏 |
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.
Now that the project is used in production, we need to explain key concepts and document commands and backends.
Purpose
Now that the project is used in production, we need to explain key concepts and document commands and backends.
Proposal
write tutorialpostponed (will be handled in another PR)