Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ocaml updates #348

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

jakubsvec001
Copy link

I am grateful for all the work you've put into making this book, and for making it available in OCaml as well.

I found a few minor errors in the snippets, where the OCaml code appears to be misplaced in section 4.3 concerning Kleisli Categories.

Fortunately, section 8.5 reiterates these concepts with the correctly associated code snippets.

I kept the same formatting of the snippets in section 8.5 and placed them in the correct files.

I hope this helps, and again thank you for making this book available and accessible!

This code is copied and pasted from the previous content in "src/content/1.4/ocaml/snippet05.ml" to match the Haskell code to which it is associated
This code is the same code found in the KleisliIdentity module in section 8.5 in order to match the Haskell code to which it is associated
This is the same code found in section 8.5 in the KleisliComposition module in order to match the Haskell code to which it is associated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant