Open yawaramin opened 4 years ago
Even if we haven't reached a conclusion on whether we should write compilable snippets, I still think the changes in this PR, especially the ones that remove dependency on external libraries, are important. Thanks for fixing the order @yawaramin! I don't know when/how they got out of order
cc: @mseri @XVilka
Thanks @ArulselvanMadhavan I can work on more chapters as well, am enjoying reading through the book.
Looks much cleaner and easier to read, indeed 👍
I agree. Looks good. Thanks @yawaramin
Cheers guys, I appreciate the review 👍
Will this be merged? If so I will adapt the examples accordingly in my Reason branch, too.
Sorry for not attending to this. Are there more changes coming? Also, does this tie to the prelude discussion in #237?
@hmemcpy hi, no problem. I've made some progress on Chapter 5 locally, if there is interest I can send more as I cover more chapters. It will be a good distraction for me from constantly obsessing over the news :-)
I think a distraction would be welcomed for everybody!
What is the current status of this MR? Is there a way to help? What about the other chapters?
@hmemcpy would you prefer this PR to cover the whole book? Or separate PRs for a few chapters at a time?
Hey all! So sorry for neglecting the PR! Would love to merge it. Can any of you verify there are no more changes? Unless you'd like to contribute the rest of the chapters as well?
Hi, what's the status of this PR ?
@yawaramin I'm reading the book right now, and I'm happy to help if you're still working on this. If not, I can take it from here and revise the rest of the chapters.
@hyphenrf hey that would be great, please feel free. I agree with all your comments btw.
Hi all,
What is the status of this PR ?
Thanks
hi @drupol, I haven't finished reading the book. I have notes written all over the OCaml impl, and once I've finished the book, I'll create a patch with all the edits and which closes this PR.
Cool, glad to see it's still alive then.
Looking forward for reviewing your pull request.
Cheers!
(%)
due to operator precedence issues