-
Notifications
You must be signed in to change notification settings - Fork 276
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
Development
: Add documentation for Aeolus service
#8133
Conversation
Development
: add aeolus documentationDevelopment
: Add documentation for Aeolus service
WalkthroughThe recent update enhances the Arte application by integrating the Aeolus service, which provides a REST API for generating and customizing build plans for programming exercises within the Artemis platform. This enhancement enables seamless integration with various CI systems, supports the translation of Bamboo build plans, and expands customization options for programming exercises across all supported languages. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Overall documentation looks good and is clear apart from the ‘API Keys’ section. I’ve left some smaller suggestions.
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
Co-authored-by: Benedikt Fein <fein@fim.uni-passau.de>
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 have some minor suggestions. Other than that lgtm
Co-authored-by: Laurenz Blumentritt <38919977+laurenzfb@users.noreply.github.com>
Co-authored-by: Laurenz Blumentritt <38919977+laurenzfb@users.noreply.github.com>
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.
Looks good, thanks for adding the documentation :)
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.
LGTM!
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.
lgtm after changes
Checklist
General
Motivation, Description and Context
Adding some documentation on how to setup aeolus as a service for Artemis to use.
https://artemis-platform--8133.org.readthedocs.build/en/8133/dev/setup/aeolus.html
https://artemis-platform--8133.org.readthedocs.build/en/8133/admin/setup/aeolus.html
Steps for Testing
read the newly added docs
Review Progress
Code Review
Summary by CodeRabbit