cs50 / checks

Checks for check50
17 stars 34 forks source link

update mario checks to look for trailing whitespace and advise accordingly #17

Open dmalan opened 6 years ago

dmalan commented 6 years ago

It would be handy to mention on the guidelines that you don't need to add spaces on the other side of the right hand column to make the spaces and pound signs symmetrical (i.e., that it's ragged right).

screen shot 2017-09-16 at 22 04 33

brianyu28 commented 6 years ago

We actually already do this, per:

https://github.com/cs50/checks/blob/master/cs50/2017/fall/mario/less/check50/__init__.py#L76

See the detailed link in the screenshot for what that looks like:

https://cs50.me/checks/e9a0cb104971aa13e0370849151e1660ee7bfaf9

dmalan commented 6 years ago

Ah, hm, so it's the abbreviated output that's referred to. Should we perhaps highlight spaces in the web output to make things more clear?