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

Fix issue typelevel/cats-effect#403 #762

Merged
merged 6 commits into from Nov 2, 2018

Conversation

Projects
None yet
1 participant
@alexandru
Member

alexandru commented Nov 2, 2018

Related: typelevel/cats-effect#403

Also added AssignableCancelableF and SingleAssignCancelableF, used in the initial implementation, which was later optimized to do without.

alexandru added some commits Nov 2, 2018

@codecov

This comment has been minimized.

codecov bot commented Nov 2, 2018

Codecov Report

❗️ No coverage uploaded for pull request base (master@9d7505f). Click here to learn what that means.
The diff coverage is 90.41%.

@@            Coverage Diff            @@
##             master     #762   +/-   ##
=========================================
  Coverage          ?   90.37%           
=========================================
  Files             ?      418           
  Lines             ?    11833           
  Branches          ?     2149           
=========================================
  Hits              ?    10694           
  Misses            ?     1139           
  Partials          ?        0

alexandru added some commits Nov 2, 2018

@alexandru alexandru merged commit 30fa29c into monix:master Nov 2, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment