Added traverse and sequence functions for both Option and AsyncOption types. I made this due to discussion in the discord channel and thought it may be pretty straight forward.
Functions added:
List.traverseOptionM
List.traverseAsyncOptionM
List.sequenceOptionM
List.sequenceAsyncOptionM
Types of changes
[X] New feature (non-breaking change which adds functionality)
Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.
[X] Build and tests pass locally
[X] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)
Further comments
If the behavior matches what is expected, let me know and I can create the documentation as well 😄
Do we think the applicative versions should be added as well? I didn't see an immediate use case but I know pblasucci in the discord was the one with the use case
Proposed Changes
Added traverse and sequence functions for both Option and AsyncOption types. I made this due to discussion in the discord channel and thought it may be pretty straight forward.
Functions added:
Types of changes
Checklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Further comments
If the behavior matches what is expected, let me know and I can create the documentation as well 😄
Do we think the applicative versions should be added as well? I didn't see an immediate use case but I know
pblasucci
in the discord was the one with the use case