search

OCamlverse / ocamlverse.github.io

Documentation of everything relevant in the OCaml world
https://ocamlverse.net
Creative Commons Zero v1.0 Universal
176 stars 67 forks source link

Add Concurrency and Parallelism with OCaml #185

Closed omasanori closed 1 year ago

gasche commented 1 year ago

Drive-by feedback: I always prefer text documents to videos; when you point to a URL, could you add (video) in the description when it is actually a video, so that I know to prefer non-video links?

omasanori commented 1 year ago

Drive-by feedback: I always prefer text documents to videos; when you point to a URL, could you add (video) in the description when it is actually a video, so that I know to prefer non-video links?

Agreed. I will add the indicator and add links to conference papers or slides if they are published. Thanks for feedback, @gasche !

gasche commented 1 year ago

Thanks! Another drive-by comment: I see that you have extended your PR with some extra topics to discuss. I would recommend instead to get the PR merged with the initial scope of things you wanted to discuss, and then send a follow-up PR with extra material if you wish to. Smaller PRs are easier to review, and my impression of ocamlverse is that it is more of a wiki were faster iterations (and less perfectionism on each change) are the default.

omasanori commented 1 year ago

Another drive-by comment: I see that you have extended your PR with some extra topics to discuss. I would recommend instead to get the PR merged with the initial scope of things you wanted to discuss, and then send a follow-up PR with extra material if you wish to. Smaller PRs are easier to review, and my impression of ocamlverse is that it is more of a wiki were faster iterations (and less perfectionism on each change) are the default.

Indeed. I will do so. Thanks!