-
Notifications
You must be signed in to change notification settings - Fork 23
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
Configuration via YAML #164
Comments
Hi! I'd like to try implementing this feature. Can I be assigned to this issue? |
Hey, @Idkwhoami42, go for it! |
Hi, I had a question regarding type safety of the YAML file. Should I implement proper type safety and checks (with specific errors messages). |
Great question, @Idkwhoami42! I don't think we need a full-blown super detailed type checker, but if anything is wrong, we should stop Andy and let the teacher know. Any extra hint to identify the problem in the exception message is welcome! |
Hi, @Idkwhoami42, are you still working on this one? |
Hi, Sorry for the bad communication, I was quite busy with exams and travelling.. but now I can start working on it again. I had implemented most of it before so shouldn't take that long hopefully. Sorry for the delay! |
Awesome!! |
Configurations are provided to Andy via a Java file that we, as teachers, have to write. The idea of making it into a Java class is that we'd get the support of the compiler to know if something is wrong. In practice, we never write these files in an IDE. It's always in WebLab. And because of that, the verbosity of Java is cumbersome.
We should support configuration via YAML files.
Suggestion:
RunConfiguration
.RunConfiguration
RunConfiguration
.The text was updated successfully, but these errors were encountered: