kfl / staffeli_nt

Staffeli NT Technology
MIT License
8 stars 7 forks source link

"Sane" error reporting wrt. to feedback-files #18

Closed madsobitsoe closed 2 years ago

madsobitsoe commented 3 years ago

When a grade.yml file is formatted incorrectly and run through upload.py, staffeli_nt currently crashes with an uncaught exception. There is no direct info on what caused the crash and chasing down the syntax-mistake in ~20 .yml files is tedious.

I propose a bit of error-handling. Catching exceptions from parsing and printing the path of the culprit-file would be a great start.

kfl commented 3 years ago

With PR #28 merged, I think that the error reporting is no longer piss-poor. Maybe it is even "sane".

@madsobitsoe is it OK that we close this issue?

madsobitsoe commented 2 years ago

Yes, very ok!