Veranderen van databestanden bij adventure kan leiden tot crashes/falende checks

[Jelleas]

Zo vertrouwde een uitwerking erop dat er een lege regel stond aan het einde van synonyms.dat, maar dat stond natuurlijk niet zo bij check50.

Misschien een idee om even te checken of de data bestanden van de student, anders zijn dan de databestanden bij de checks?

[Jelleas]

Yup echt even checken! Debuggen is onmogelijk als de databestanden niet overeenkomen

[bjorn-out]

Afgezien van minprog/cs50x#42 zijn er geen afwijkingen (hiervoor zijn nu PR's aangemaakt). De databestanden in check50 hebben ook lege regels aan het eind.

[Jelleas]

Studenten passen de bestanden weleens aan :)!

Misschien slim om, als de bestanden bestaan, ze mee te nemen in de check. Dan vervolgens in de eerste check even na te gaan of deze niet anders zijn dan de bestanden bij de checks zelf.

[bjorn-out]

Oh, dat bedoelde je daarmee. Ja, dat kan inderdaad.
Dan moeten ze wel ook mee ingeleverd worden via de website (anders zijn die bestanden niet beschikbaar)

[Jelleas]

Mijn idee zou zijn om ze enkel mee te nemen als ze bestaan, ofwel in .cs50.yaml

check50:
    files:
        - !require "adventure.py"
        - !require "room.py"
        - !include "SmallAdv.dat"
        - !include "TinyAdv.dat"

Als ze bestaan, kunnen we even kijken of ze afwijken en vervolgens op de afwijking wijzen.