-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
3.x: Refactor to io.reactivex.rxjava3.** #6621
Conversation
Codecov Report
@@ Coverage Diff @@
## 3.x #6621 +/- ##
======================================
Coverage ? 98.19%
Complexity ? 6191
======================================
Files ? 678
Lines ? 44792
Branches ? 6173
======================================
Hits ? 43982
Misses ? 278
Partials ? 532 Continue to review full report at Codecov.
|
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.
I trust you
You can compare lines and branch counts: #6620 (comment) @@ Coverage Diff @@
## 3.x #6620 +/- ##
============================================
+ Coverage 98.13% 98.17% +0.03%
- Complexity 6187 6188 +1
============================================
Files 678 678
Lines 44792 44792
Branches 6173 6173
============================================
+ Hits 43956 43973 +17
+ Misses 290 281 -9
+ Partials 546 538 -8 |
Go ahead and merge! |
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.
👍
@@ -52,7 +52,7 @@ targetCompatibility = JavaVersion.VERSION_1_6 | |||
// --------------------------------------- | |||
|
|||
def junitVersion = "4.12" | |||
def reactiveStreamsVersion = "1.0.2" | |||
def reactiveStreamsVersion = "1.0.3-RC1" |
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.
Prob should have been done in a separate PR, but not a big deal
This PR refactors the package structure to match the group ID:
io.reactivex.rxjava3
. In addition, the base classes and interfaces now live inio.reactivex.rxjava3.core
.In addition, the Reactive Streams dependency was bumped to 1.0.3-RC1 to verify the TCK changes.