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

Major, try to keep sync/async at the type level #179

Closed
mikearnaldi opened this issue Mar 29, 2020 · 5 comments
Closed

Major, try to keep sync/async at the type level #179

mikearnaldi opened this issue Mar 29, 2020 · 5 comments
Labels
enhancement New feature or request

Comments

@mikearnaldi
Copy link
Member

Ideally we can use a Kind4 instead of 3 keeping "a" | "s" as the effect type and combine accordingly, unsure if possible in all cases with correct inference.

@mikearnaldi mikearnaldi added the enhancement New feature or request label Mar 29, 2020
@mikearnaldi
Copy link
Member Author

WIP #180

@mikearnaldi
Copy link
Member Author

Keeping "a" | "s" is a mess type wise. I have found a better encoding placing:

type ASYNC = unknown
type SYNC = never

Such that: ASYNC | SYNC = ASYNC

@mikearnaldi
Copy link
Member Author

This is released, to be considered still a work in progress, especially in the sense that Stream, Managed & other apis might benefit from the same kind of refinement.

Probably the end goal is to have an eff package containing all the api in this form with a mapping to effect

@mikearnaldi
Copy link
Member Author

#195 Hit a wall on the current design

@mikearnaldi
Copy link
Member Author

done in #199

fubhy pushed a commit that referenced this issue Jan 1, 2024
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

No branches or pull requests

1 participant