Skip to content

Drop RxJava 2#540

Merged
rschmitt merged 1 commit intomasterfrom
rxjava3
Jul 29, 2025
Merged

Drop RxJava 2#540
rschmitt merged 1 commit intomasterfrom
rxjava3

Conversation

@rschmitt
Copy link
Copy Markdown
Contributor

RxJava 2 is end-of-life as of February 28, 2021. ReactiveTestUtils can probably be dropped for 5.6 in favor of Reactive3TestUtils.

RxJava 2 is end-of-life as of February 28, 2021. `ReactiveTestUtils` can
probably be dropped for 5.6 in favor of `Reactive3TestUtils`.
Comment thread pom.xml
<parameter>
<breakBuildOnBinaryIncompatibleModifications>true</breakBuildOnBinaryIncompatibleModifications>
<breakBuildOnSourceIncompatibleModifications>true</breakBuildOnSourceIncompatibleModifications>
<ignoreMissingClasses>true</ignoreMissingClasses>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@rschmitt This is due to formatting, is it not?

Copy link
Copy Markdown
Contributor Author

@rschmitt rschmitt Jul 29, 2025

Choose a reason for hiding this comment

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

No, I changed this by hand. This option appeared to be in the wrong section and wasn't taking effect. Without this option here, japicmp fails because the current version's classpath is missing the RxJava 2 classes (not to be confused with the missing ReactiveTestUtils class). Evidently, ignoreMissingClasses needs to be set in the build plugin config, not the reporting plugin config.

@rschmitt rschmitt merged commit 4b9db72 into master Jul 29, 2025
17 checks passed
@rschmitt rschmitt deleted the rxjava3 branch July 29, 2025 16:39
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.

2 participants