Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update type refactoring #407

Open
flandweber opened this issue Jun 29, 2024 · 0 comments
Open

Update type refactoring #407

flandweber opened this issue Jun 29, 2024 · 0 comments

Comments

@flandweber
Copy link
Contributor

flandweber commented Jun 29, 2024

as mentioned in #406 (comment) I believe the Update module needs a refactoring.
Especially the function runUpdate' seems overly complex.
I'm sure there's a better abstraction to be found, though I don't know what it is yet.
maybe System.IO.Lazy could play a role?

@nmattia what was the original reason to make Update an arrow instead of a monad?
I'm new to arrows so I don't yet fully grasp the advantages it has over monads in the implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant