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.

Sign up for GitHub

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

Jump to bottom

Support ValueOption in FsToolkit.Errorhandling.asyncOption #225

Open
Xyncgas opened this issue Aug 4, 2023 · 3 comments
Open

Support ValueOption in FsToolkit.Errorhandling.asyncOption #225

Xyncgas opened this issue Aug 4, 2023 · 3 comments
Labels
good first issue Good for newcomers

Comments

@Xyncgas
Copy link

Xyncgas commented Aug 4, 2023

currently we can use let! x = with say int option but not int voption
it seems voptions can reduce memory allocation with option types
@TheAngryByrd
Copy link
Collaborator

Yep I agree this is a good idea. Would you like to submit a pull request for it?

@TheAngryByrd TheAngryByrd added the good first issue Good for newcomers label Aug 4, 2023
This was referenced Aug 4, 2023
Merged
Closed
@Xyncgas
Copy link
Author

Xyncgas commented Aug 4, 2023

Maybe instead of making it possible by doing what OptionCE did, I can make asyncOption CE to use voption instead of converting voption to option

@Xyncgas
Copy link
Author

Xyncgas commented Aug 4, 2023

in the future

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TheAngryByrd @Xyncgas