reactive-banana icon indicating copy to clipboard operation
reactive-banana copied to clipboard

Generalise `compile`, `actuate` and `pause` to `MonadIO`

Open ocharles opened this issue 2 years ago • 3 comments

ocharles avatar Sep 16 '21 16:09 ocharles

Perhaps lift pause :: EventNetwork -> IO () too?

vagarenko avatar Sep 16 '21 18:09 vagarenko

Good shout @vagarenko - I think we should do all of these or none of them, so I'll add pause into this.

ocharles avatar Sep 16 '21 18:09 ocharles

Alas, I'm not entirely sold on the introduction of MonadIO, as it makes the types more complicated.

I have made a comment in the related pull request https://github.com/HeinrichApfelmus/reactive-banana/pull/225#issuecomment-968275468 . To avoid duplication, I propose that we discuss this issue there.

HeinrichApfelmus avatar Nov 14 '21 11:11 HeinrichApfelmus