-
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
Integrated code lifecycle
: Enhance handling of build scripts in the client
#8165
Conversation
767ac5d
to
bc80afe
Compare
WalkthroughThe updates focus on enhancing programming exercise management and build configuration within an educational platform. A significant addition is the method for generating default build scripts based on exercise properties, alongside improvements in local continuous integration service handling and user interface enhancements for displaying programming exercise details, specifically build configurations. Changes streamline the process of managing programming exercises, from build plan recreation to UI updates for a better user experience in configuring and understanding exercise build specifics. 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 (
|
61881bb
to
a178c25
Compare
a178c25
to
5d2b8d8
Compare
Development
: Enhance handling of build scripts in the clientIntegrated code lifecycle
: Enhance handling of build scripts in the client
I’m not sure if adding this as additional information to the exercise details page is a bit too much. Would it be possible to show the build script in the already existing (at least for Jenkins) build plan editor or refactor that page so that you can see either the Jenkins pipeline definition or the build script depending on the system/exercise setup? The editor could be shown in read-only mode for the build script in either case if manually editing the script is not or should not be supported. Rel #8079 |
src/main/java/de/tum/in/www1/artemis/service/connectors/BuildScriptProvider.java
Show resolved
Hide resolved
17abde3
to
c27f70f
Compare
Hi @b-fein! Thanks for the questions. I think this would result in a much bigger PR with a lot of refactoring that maybe could be done at a later time, for now this is a solution for LocalCI only and not Jenkins. |
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.
Some minor comments, looks good :)
Co-authored-by: Raphael Stief <118574504+rstief@users.noreply.github.com>
src/main/webapp/app/exercises/programming/manage/update/programming-exercise-update.module.ts
Outdated
Show resolved
Hide resolved
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.
Thanks for the changes, 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.
tested in testing session
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.
Demonstrated in testing session. LGTM
Checklist
General
Server
Client
Changes affecting Programming Exercises
Motivation and Context
I added the build script to the detail page to make it easier to check what will be run, it is now also possible to reset the build plan back to the template using the reset button which was already there. I further improved some translations and applied the new guideline, and removed some unused services.
Description
Instructors can now quickly see their build script and revert to the template if something does not work
Steps for Testing
Prerequisites:
Testserver States
Note
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Review Progress
Performance Review
Code Review
Manual Tests
Test Coverage
Client
Server
Screenshots
Summary by CodeRabbit
Summary by CodeRabbit
artemisTranslate
pipe for localization, improving consistency and removing deprecated elements.