fmidue / haskell-template-task

Other
0 stars 0 forks source link

Display issues in feedback #8

Closed jvoigtlaender closed 5 months ago

jvoigtlaender commented 5 months ago

First, this is not really nice (even if just since it defeates simple copy-and-paste of a submission into another editor): image Couldn't there at least be a line break after the gelesen:?

Second, the path printout here is quite distracting: image

Probably something like this function should be applied there: https://github.com/fmidue/haskell-template-task/blob/0d393c806d5fa571e640e277a2e8ce62b6dafdd9/src/Haskell/Template/Task.hs#L401-L404

marcellussiegburg commented 5 months ago

Note: this change might make it harder to find actual configuration errors (as no module name is shown now whatsoever; i.e. the user has to figure out, if the error is in the program, the tests or any of the additional programs). If better guidance might be required, the fix could be amending editFeedback, printing the characters after the last /. This would be a separate issue.

jvoigtlaender commented 5 months ago

Die Sache mit

Couldn't there at least be a line break after the gelesen:?

ist auch ein anderes Issue? Vielleicht sogar Autotool-seitig?

marcellussiegburg commented 5 months ago

Ja, genau. Das ist Autotool seitig und sollte mit dem vermutlich heute erfolgenden Deployment auch behoben werden. (Der entsprechende commit wird entsprechend auch im Autotool-Repository hinterlegt.)