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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add RxJava 3 module #1929
Add RxJava 3 module #1929
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.
Great work!
Please rebase your branch on top of develop
and target develop
branch as well, our master
reflects the state of the latest stable release
Coroutine builders: | ||
|
||
| **Name** | **Result** | **Scope** | **Description** | ||
| --------------- | --------------------------------------- | ---------------- | --------------- |
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.
Please fix the packages in this file in order to make knit task pass without warnings
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.
Done - f44e6b5. I couldn't find which task in the knit task to run to test this locally though
This isn't the most obvious but I guess it's intentional "I know what I'm doing" code :)
Rebased and changed the target to |
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.
Resolves #1883
I took the liberty of omitting deprecated rx2 APIs from this migration, but it wasn't entirely possible without using
@PublishedApi
for a couple of places.There's a decent amount of code duplication involved with this. Could consolidate some for test utilities, but not sure how worth it it is if the rx2 artifact isn't going to see much further development. Let me know 馃憤