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

Add onErrorClose #371

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add onErrorClose #371

wants to merge 5 commits into from

Conversation

brianegan
Copy link
Collaborator

Add onErrorClose to the onErrorResume extension methods. Fixes #369

* Extension Methods phase 1

  - Introduce ObservableExtensions
  - Move all Rx operators from Observable to *Extension
  - Remove all "wrapper" methods from Observable
    - Default to Stream Implementations
  - Keep constructors on Observable class
  - Remove Exported Futures & Tests
    - AsObservable Future no longer provides utility
    - Min and Max are easily inlined
  - Remove OfType
  - Remove concatMap
  - Add documentation for all Stream extensions
@brianegan brianegan changed the base branch from master to extension-methods November 22, 2019 16:27
@codecov-io
Copy link

codecov-io commented Nov 22, 2019

Codecov Report

Merging #371 into master will decrease coverage by 0.2%.
The diff coverage is 90.13%.

@@            Coverage Diff             @@
##           master     #371      +/-   ##
==========================================
- Coverage   93.35%   93.14%   -0.21%     
==========================================
  Files          59       60       +1     
  Lines        2016     2057      +41     
==========================================
+ Hits         1882     1916      +34     
- Misses        134      141       +7

@shinayser
Copy link

Nice job guys!! Any Idea when we getting it in stable?

@brianegan brianegan changed the base branch from extension-methods to master December 17, 2019 15:55
@brianegan
Copy link
Collaborator Author

Heya @shinayser -- a bit of a busy time for me with holidays coming up. I'll hopefully get to it over the break, but no promises.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add onErrorClose to onErrorResume extensions
3 participants