Skip to content

2.x: add new methods to Maybe, Observable and Single from 4481 #4488

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

Merged
merged 2 commits into from
Sep 6, 2016

Conversation

akarnokd
Copy link
Member

@akarnokd akarnokd commented Sep 6, 2016

This PR adds just the new methods from #4481 to allow a much cleaner change of return types later on.

@akarnokd akarnokd added this to the 2.0 RC 3 milestone Sep 6, 2016
@@ -701,6 +701,34 @@
// ------------------------------------------------------------------

/**
* Waits in a blocking fashion until the current Maybe signals a success value (which is returned) or
* defaultValue if completed or an exception (which is propagated).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

null, not defaultValue

@JakeWharton
Copy link
Contributor

👍

@codecov-io
Copy link

codecov-io commented Sep 6, 2016

Current coverage is 77.79% (diff: 8.16%)

Merging #4488 into 2.x will decrease coverage by 0.36%

@@                2.x      #4488   diff @@
==========================================
  Files           497        500     +3   
  Lines         33772      33909   +137   
  Methods           0          0          
  Messages          0          0          
  Branches       5303       5325    +22   
==========================================
- Hits          26396      26378    -18   
- Misses         5409       5554   +145   
- Partials       1967       1977    +10   

Powered by Codecov. Last update 928e437...344af8d

@akarnokd
Copy link
Member Author

akarnokd commented Sep 6, 2016

Updated with corrections.

@akarnokd akarnokd merged commit ff3c5d0 into ReactiveX:2.x Sep 6, 2016
@akarnokd akarnokd deleted the MaybeOps906_1 branch September 6, 2016 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants