-
Notifications
You must be signed in to change notification settings - Fork 454
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 AsyncContext typeclass #192
Conversation
Codecov Report
@@ Coverage Diff @@
## master #192 +/- ##
=========================================
Coverage ? 53.61%
Complexity ? 201
=========================================
Files ? 94
Lines ? 1977
Branches ? 235
=========================================
Hits ? 1060
Misses ? 816
Partials ? 101
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💥
* Make some more functions inline with crossinline lambdas * Deprecate Validated.leftMap in favour of mapLeft * Deprecate Either.catch(fe, f) in favour catch(f).mapLeft(fe) * Add missing tests to Either.catch * Workaround a possible false positive from animal sniffer Co-authored-by: Simon Vergauwen <[email protected]>
This PR add AsyncContext based off Freestyle's. It includes a couple of constructor helpers to be used in comprehensions, along with basic laws.