-
Notifications
You must be signed in to change notification settings - Fork 226
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
Migrate Completable support to fully use as() #191
Migrate Completable support to fully use as() #191
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good start! Can you make the existing CompletableScoper
just delegate to AutoDispose.autoDisposable
APIs as part of this?
import io.reactivex.CompletableObserver; | ||
import io.reactivex.Maybe; | ||
|
||
class AutoDisposeCompletable extends Completable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make this final
@@ -0,0 +1,37 @@ | |||
/* | |||
* Copyright (c) 2017. Uber Technologies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2018
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
This is the CompletableConverter/CompletableScoper step of #188.
Converting to the new AutoDisposeCompletable and changes AutoDispose to utilize the new AutoDisposeCompletable.