Skip to content
This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

Abilities tutorial: exercise answers for map, filter, take #267

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stefanholzmueller
Copy link

  • The answer for the take exercise was missing.
  • map and filter were only implemented in terms of pipe (which is a "very challenging" exercise) and therefore made the tutorial hard to follow, IMHO. So I added simpler implementations and renamed the existing ones.
  • Moved the answer for sum' a bit up, reflecting the order of the exercises in the tutorial.

I implemented the answers myself, hopefully in an idiomatic way. The function signatures in the base library slightly differ from the tutorial exercises and their implementation is not straightforward, so I did not use those.

I could not find any contribution guidelines... I just worked through the exercises and noticed the answers could be improved. If this PR is not very useful, feel free to reject it. I am also happy to receive feedback how to change it. Oh and I am not sure which branch to compare the PR against, is master correct?

Note that the solution for take using pipe is still missing. I couldn't come up with one. Very challenging indeed. ;) Otherwise an excellent, fun tutorial!

* The answer for the `take` exercise was missing.
* `map` and `filter` were only implemented in terms of `pipe` (which is a "very challenging" exercise) and therefore made the tutorial hard to follow, IMHO. So I added basic implementations and renamed the existing ones.
* TODO: implement `take'` in terms of `pipe`
@netlify
Copy link

netlify bot commented Sep 6, 2021

👷 Deploy request for unisonweb-org pending review.
Visit the deploys page to approve it

🔨 Explore the source changes: e2762d3

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant