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

Lambda Hacknight Changes #122

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

7h3kk1d
Copy link
Member

@7h3kk1d 7h3kk1d commented Jun 10, 2022

I'll split this up into multiple PRs. Mostly creating this draft as a springboard for discussion.

Topics addressed:

  • Override checkedApply on SemigroupFactory to help resolve checkedApply/apply incoherence
  • Introduced a ShortCircuitingSemigroup with the hope of making short circuiting an easier pattern to add to semigroups/monoids in the future.
    • Does not address foldRight.
    • I'm not sure this is a good idea or if there's a better way to incorporate laziness
  • Make collapse short circuit if it's underlying semigroups do

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

Successfully merging this pull request may close these issues.

1 participant