Add relative-distance
#752
Conversation
jiegillet
left a comment
jiegillet
left a comment
There was a problem hiding this comment.
Oh yeah, I remember writing down that list was a pain :D
Well done, nice work!
exercises/practice/relative-distance/.meta/src/RelativeDistance.example.elm
Outdated
Show resolved
Hide resolved
exercises/practice/relative-distance/.meta/src/RelativeDistance.example.elm
Outdated
Show resolved
Hide resolved
exercises/practice/relative-distance/.meta/src/RelativeDistance.example.elm
Outdated
Show resolved
Hide resolved
config.json
Outdated
| "maybe" | ||
| ], | ||
| "prerequisites": [ | ||
| "dict" |
There was a problem hiding this comment.
Let's just practice dict on this one.
For prerequisites, let's add:
- maybe
- lists
- set
- recursion
- pattern matching
maybe tail-call-recursion once #751 is sorted out?
There was a problem hiding this comment.
The prerequisites seem fine. However, this seems solvable without knowing sets.
There was a problem hiding this comment.
Your solution uses sets. And in general, I would suggest for students to know about sets before they try this one, because it can really help.
There was a problem hiding this comment.
I added everything but recursion since I didn't see it in the repo as an existing concept. I wasn't sure if that meant students couldn't unlock the exercise until that concept is added.
There was a problem hiding this comment.
It was added after you opened your PR. If you merge main now, you'll see it.
jiegillet
left a comment
jiegillet
left a comment
There was a problem hiding this comment.
Looks good, thank you again!
If you could merge main and add the recursion prerequisite, that would be great!
BNAndras
force-pushed
the
add-relative-distance
branch
from
2c48061 to
cb363ba
Compare
3 participants
Closes #739. Building up the graph of connections took a while :)