-
Notifications
You must be signed in to change notification settings - Fork 450
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
Add ApplicativeError laws #236
Conversation
Codecov Report
@@ Coverage Diff @@
## master #236 +/- ##
=========================================
Coverage ? 49.57%
Complexity ? 232
=========================================
Files ? 109
Lines ? 2453
Branches ? 287
=========================================
Hits ? 1216
Misses ? 1124
Partials ? 113
Continue to review full report at Codecov.
|
Can this be merged now ? |
I need to fix the conflicts first. Haven't had the time yet. |
# Conflicts: # kategory-effects/src/test/kotlin/kategory/effects/data/JobKWTest.kt
This PR adds the laws for ApplicativeError. They add another EQ to the call chain, it wasn't too painful. Depends on #234, see diff in #235