search

ls1intum / Artemis

Artemis - Interactive Learning with Automated Feedback
https://docs.artemis.cit.tum.de
MIT License
477 stars 284 forks source link

`Online-Editor / Java-programming-exercises`: Hide pom.xml in Online-Editor #5046

Open Shirkanesi opened 2 years ago

Shirkanesi commented 2 years ago

Is your feature request related to a problem?

As a follow-up to https://github.com/ls1intum/Artemis/pull/4947 I'd suggest to hide the pom.xml in the Online-Editor. From my point of view this file will confuse unexperienced students (as those are the ones actually using the online-editor) quite a lot. This confusion will lead to a lot of unneccessary quiestions by the students which can not realy be answered in a satisfying way without explaining the entire concept behind maven (which cannot realy be done either, if the student heared the first time about java just two weeks before...).

Also, as far as I understood the changes in the above mentioned PR, the pom.xml is only there to easily import the project into an IDE, so there is no reason to have it in the Online-Editor anyway.

Please correct me if I'm totaly missing anything about it :)

Describe the solution you'd like

Hide the pom.xml in the editor.

Describe alternatives you've considered

Keep the pom.xml in the editor but at least add a comment on top of it saying this file should not be modified.

Additional context

No response

krusche commented 2 years ago

I agree that we should hide this for exercises "without dependencies". If the exercise requires the students to use dependencies, we should display it. I think we can decide this based on the chosen project type

bassner commented 2 years ago

However, it's still possible to add dependencies to projects created by the template without an exemplary dependency ... It could probably just be a checkbox in a programming exercise (e.g. "show maven / gradle files in online editor") ..?