search

SERG-Delft / andy

Andy assesses student's test code. It's used in CSE1110, TU Delft.
MIT License
78 stars 22 forks source link

Allow class names to include "Configuration" #243

Open angelikamentz opened 1 year ago

angelikamentz commented 1 year ago

Right now, when a class name contains the word "Configuration" a RuntimeException is thrown on the configuration being null. We want to fix this bug.

mauricioaniche commented 1 year ago

Great! I don't think we should make it work, but more like return a nice message to the teacher, like, "hey, Configuration is reserved, you can't use it".

(We should open another issue to rename this class. "Rubric" would be much better!)