Skip to content
This repository has been archived by the owner on Feb 24, 2021. It is now read-only.

Update README.MD #251

Merged
merged 5 commits into from Aug 13, 2020
Merged

Update README.MD #251

merged 5 commits into from Aug 13, 2020

Conversation

PhBastiani
Copy link
Contributor

Additionnal notes : Atomic & ConcurrenVar

Additionnal notes : Atomic & ConcurrenVar
@nomisRev
Copy link
Member

Thanks @PhBastiani!

This is great 👏 👏 No work on this has been done yet, but I proposed something in #248 which is preparing the docs for release. I'd love your feedback there as well!

This could become a page that in more detail explains the integrations and cancellation in the standard library, etc.
A fourth page could be added to the quickstart which covers the IO vs suspend section currently in the README.MD of Arrow Fx Coroutines. It could also include a small migration guide from IO to suspend as today was asked on Slack

Copy link
Member

@nomisRev nomisRev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we mention that the other concurrency data types are still available as well, but with the same names? We also added a new data type which is CircuitBreaker.

All of these will also be promoted and covered in the Arrow Fx websites sidebar. Link to Arrow Site PR

arrow-fx-coroutines/README.MD Show resolved Hide resolved
@rachelcarmena rachelcarmena merged commit 10fafc3 into arrow-kt:master Aug 13, 2020
@rachelcarmena
Copy link
Member

Global checks were cancelled because they aren't necessary. Thanks @PhBastiani !!

@PhBastiani PhBastiani deleted the patch-1 branch August 13, 2020 12:21
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.

None yet

3 participants