This PR adds back the main EventBus dependency, which is Android related.
This dependency was incorrectly removed and replaced by its corresponding eventbus-java transitive dependency, assuming that is not required. However, after further testing it was revealed that this parent eventbus dependency, which is actually android specific is indeed necessary. If it is not provided on the classpath then an app will crash with the below runtime exception:
java.lang.RuntimeException: Unable to create application xyz: java.lang .RuntimeException:
It looks like you are using EventBus on Android, make sure to add the "eventbus" Android
library to your dependencies.
Run the ./gradlew buildHealth task and verify that there is no unused related advise related to eventbus.
Testing:
Smoke test the FluxC Example app, try out all available screens and functionality. Verify everything is working as expected.
Also, if you want to be thorough about reviewing the changes, you could quickly smoke test, either the JP/WPAndroid and/or WCAndroid, with this version of FluxC, or via composite builds, and see if it works as expected.
Depends On:
Description
This PR adds back the main
EventBusdependency, which isAndroidrelated.This dependency was incorrectly removed and replaced by its corresponding
eventbus-javatransitive dependency, assuming that is not required. However, after further testing it was revealed that this parenteventbusdependency, which is actuallyandroidspecific is indeed necessary. If it is not provided on the classpath then an app will crash with the below runtime exception:For more info see:
Testing Instructions
./gradlew buildHealthtask and verify that there is nounusedrelated advise related toeventbus.FluxC Exampleapp, try out all available screens and functionality. Verify everything is working as expected.FluxC, or via composite builds, and see if it works as expected.