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

Upgrade cats (issue #212), optimize Task, refactorings #214

Merged
merged 5 commits into from Aug 25, 2016

Conversation

Projects
None yet
1 participant
@alexandru
Copy link
Member

alexandru commented Aug 25, 2016

  • upgrade the Cats dependency to version 0.7.0, see issue #212
  • rename eval to evalAlways across the board (in Task, Coeval and Observable), but keep evalAlways with the @deprecated sign
  • rename eval(Coeval) to coeval(Coeval) in Task and Observable
  • refactor the Task internals again, for optimizations
    • simplifies the internal states, e.g. instead of having Now, Error, Always and Once, we now have a single Delay(coeval)
    • get rid of Task.Attempt, it never made any sense that one. People can use Coeval.Attempt if they need a Try alternative (and convert to Task if they end up needing a Task)
  • moved everything from monix.types.shims to monix.types
  • in monix.types, split the Deferrable type in Suspendable and Memoizable
  • in monix-execution introduce executeAsync and executeLocal as macros
  • use executeLocal and LocalRunnable in key points in the Task implementation to reduce forking

alexandru added some commits Aug 23, 2016

@alexandru alexandru merged commit ec85693 into master Aug 25, 2016

0 of 2 checks passed

continuous-integration/travis-ci/push The Travis CI build failed
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@alexandru alexandru deleted the wip-refactorings-2 branch Nov 8, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.