Skip to content

Conversation

akarnokd
Copy link
Member

This PR refactors the package structure to match the group ID: io.reactivex.rxjava3. In addition, the base classes and interfaces now live in io.reactivex.rxjava3.core.

In addition, the Reactive Streams dependency was bumped to 1.0.3-RC1 to verify the TCK changes.

@akarnokd akarnokd added the 3.x label Aug 16, 2019
@akarnokd akarnokd added this to the 3.0 milestone Aug 16, 2019
@codecov
Copy link

codecov bot commented Aug 16, 2019

Codecov Report

❗ No coverage uploaded for pull request base (3.x@f8841c7). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@ 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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f8841c7...5adb308. Read the comment docs.

Copy link
Collaborator

@vanniktech vanniktech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I trust you

@akarnokd
Copy link
Member Author

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
@vanniktech
Copy link
Collaborator

Go ahead and merge!

@akarnokd akarnokd merged commit a1693ec into ReactiveX:3.x Aug 16, 2019
@akarnokd akarnokd deleted the 3xMovePackage branch August 16, 2019 12:51
Copy link
Contributor

@artem-zinnatullin artem-zinnatullin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍


def junitVersion = "4.12"
def reactiveStreamsVersion = "1.0.2"
def reactiveStreamsVersion = "1.0.3-RC1"
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants