Run CI Tests on Windows

SERG-Delft / andy

Andy assesses student's test code. It's used in CSE1110, TU Delft.

MIT License

78 stars 23 forks source link

Run CI Tests on Windows #263

Closed alexcojocaru2002 closed 4 months ago

alexcojocaru2002 commented 1 year ago

Closes #218.

Closes #253.

The aim of this PR is to add windows as an OS option when running the CI and to fix it in case the previous errors are still existent.

martinmladenov commented 1 year ago

@alexcojocaru2002 Regarding #218: Did it turn out there was no bug after all? Does the temp directory get properly deleted?

alexcojocaru2002 commented 1 year ago

@alexcojocaru2002 Regarding #218: Did it turn out there was no bug after all? Does the temp directory get properly deleted?

I think the bug fixed itself with the new additions to the workflow files... I can't see the logs anymore so I am not sure which temporary file it was, but I would assume yes since the actions are running fine.

martinmladenov commented 1 year ago

OK, I'll close the issue then. Thanks!