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

2.x: coverage, fixes, cleanup 10/21-1 #4742

Merged
merged 1 commit into from
Oct 21, 2016

Conversation

akarnokd
Copy link
Member

  • fix Flowable.materialize() terminal signal emission in face of backpressure
  • fix Flowable.onErrorReturn() terminal signal emission in face of backpressure
  • cleanup Flowable.publish() and enable operator fusion on its input
  • fix Flowable.flatMapSingle() and Flowable.flatMapMaybe() termination detection
  • compact Flowable.blockingSubscribe() and Observable.blockingSubscribe()
  • compact Flowable.subscribeOn()

@akarnokd
Copy link
Member Author

There seems to be a problem between GitHub and Travis at the moment.

@akarnokd akarnokd closed this Oct 21, 2016
@akarnokd akarnokd reopened this Oct 21, 2016
@codecov-io
Copy link

codecov-io commented Oct 21, 2016

Current coverage is 93.95% (diff: 93.15%)

Merging #4742 into 2.x will increase coverage by 0.40%

@@                2.x      #4742   diff @@
==========================================
  Files           569        570     +1   
  Lines         37148      37064    -84   
  Methods           0          0          
  Messages          0          0          
  Branches       5666       5648    -18   
==========================================
+ Hits          34750      34822    +72   
+ Misses         1271       1136   -135   
+ Partials       1127       1106    -21   

Powered by Codecov. Last update 14a954c...e3ca3ee

@akarnokd akarnokd merged commit 98d0b7f into ReactiveX:2.x Oct 21, 2016
@akarnokd akarnokd deleted the Coverage_10_21_1 branch October 21, 2016 15:47
@akarnokd akarnokd mentioned this pull request Oct 27, 2016
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.

3 participants