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

Clean up some duplicate functions and rename some #17

Merged
merged 9 commits into from
Dec 8, 2021
Merged

Clean up some duplicate functions and rename some #17

merged 9 commits into from
Dec 8, 2021

Conversation

jamil7
Copy link
Owner

@jamil7 jamil7 commented Dec 8, 2021 •
edited
Loading

  • Remove the functions that have an Applicative and a Monadic instace that work exactly the same, and default to use the monadic one while keeping an applicative version in case it runs in parallel
  • Rename mapM to traverse
  • Rename sequenceAParellel to only parallel
  • Remove all sequencing and traversing from List module
  • Support net6.0 only

@jamil7 jamil7 self-assigned this Dec 8, 2021
@jamil7 jamil7 merged commit 9d2526d into main Dec 8, 2021
@jamil7 jamil7 deleted the remove-some-funcs branch December 8, 2021 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jamil7 jamil7

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jamil7