search

gradescope / gradescope-utils

Python package for building Gradescope autograders
https://gradescope-utils.readthedocs.io/en/latest/
MIT License
33 stars 25 forks source link

Use "status" prop and other misc improvements #27

Closed isaacl closed 2 years ago

isaacl commented 2 years ago

Use new "status" key to mark failing tests red, even if they are 0 weight.

isaacl commented 2 years ago

@ibrahima ^

ibrahima commented 2 years ago

Hi @isaacl, thanks for your contribution! We'll have to take a look at it in the near future.

As a broad note, since this is the "default" public library for Gradescope autograders, we do need to make sure that any changes make sense for everybody, so we will be thinking about these changes from that perspective.

isaacl commented 2 years ago

Also I think we still want "if output" even if output is a str

It avoids a double newline if output string is empty.

ibrahima commented 2 years ago

Also I think we still want "if output" even if output is a str

It avoids a double newline if output string is empty.

Ah good point, I didn't realize an empty string was Falsey in Python. I thought it was just None, 0, and False.