Skip to content

Commit

Permalink
Cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
durban committed May 22, 2024
1 parent 621134c commit 513f1f7
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,6 @@
- is it usable outside of `choam`?
- if not, it doesn't really make sense to have it in a separate module(?)
- being in the same module would simplify using `ThreadContext` for `Rxn`-things
- Rename `flatMapF`
- maybe `semiFlatMap` (or `semiflatMap`?)
- or `subflatMap`?
- `Rxn.delay` use cases:
- allocating:
- `Ref` (most others are built on this)
Expand All @@ -85,19 +82,9 @@
- `Clock`
- `cats.effect.std.Random`
- `Ttrie` (to avoid `Rxn`-level contention)
- Maybe rename `Ref`?
- Collision with `cats.effect.kernel.Ref`
- Although it is hard to confuse them
- Name ideas:
- `RVar` (like `TVar` for STM refs)
- `RRef`
- ???
- Handling errors?
- Generally, we shouldn't raise errors in a reaction
- If something can fail, return `Option` or `Either`
- Need to review and document the handling of exceptions
- they should fall-through, but with cleanup (if needed)
- (maybe: do not guarantee any specific behavior for now)
- Transient errors can sometimes be handled with `+` (`Choice`)
- but sometimes this can cause infinite retry
- Composition of maybe-infinitely-retrying reactions:
Expand Down

0 comments on commit 513f1f7

Please sign in to comment.