search

cs50 / checks

Checks for check50
17 stars 34 forks source link

add non_alpha check #84

Closed curiouskiwi closed 5 years ago

curiouskiwi commented 5 years ago

Added a non_alpha character test for substrings

curiouskiwi commented 5 years ago

@brianyu28 we had no substring checks that had non-alpha chars and I've seen students code that didn't handle this properly.

brianyu28 commented 5 years ago

Good idea, thanks! The current checks we use for check50 are actually now located in the cs50/problems repo, rather than cs50/checks, so could you instead update the below two?

https://github.com/cs50/problems/blob/2019/x/similarities/__init__.py https://github.com/cs50/problems/blob/2019/ap/similarities/__init__.py

The syntax is slightly different, but should be pretty similar.