Conversation
ad463a2 to
e83b8bc
Compare
4a5b329 to
9040aa0
Compare
|
@coderabbitai review |
|
|
Warning Review limit reached
Next review available in: 59 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (30)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
9040aa0 to
4e65faa
Compare
Wire the :eppo module to use :android-sdk-framework abstractions: - EppoClient extends AndroidBaseClient<Configuration, JsonNode> - JacksonConfigurationParser: implements 2-param ConfigurationParser with parseFlagConfig + buildConfig(FlagConfigResponse, ...) - OkHttpEppoClient: implements EppoConfigurationClient interface - GsonConfigurationParser + GsonConfigurationCodec in example module Removes old ConfigurationStore, BaseCacheFile, ConfigCacheFile (moved to framework). Bumps eppo to 5.0.0-SNAPSHOT, sdk-common-jvm to 4.0.0-SNAPSHOT.
4e65faa to
f92f834
Compare
Delete cloud.eppo.android.dto.adapters/ — these were near-identical copies of cloud.eppo.ufc.dto.adapters/ from sdk-common-jvm. The :eppo module already depends on sdk-common-jvm, so import directly.
Summary
EppoClientextendsAndroidBaseClient<Configuration, JsonNode>(wasBaseEppoClientdirectly)JacksonConfigurationParser: 2-paramConfigurationParserwithparseFlagConfig+buildConfig(FlagConfigResponse, ...)OkHttpEppoClient: implementsEppoConfigurationClientinterfaceGsonConfigurationParser,GsonConfigurationCodec,CustomClientActivityConfigurationStore,BaseCacheFile,ConfigCacheFile(moved to framework):eppoto 5.0.0-SNAPSHOT,sdk-common-jvmto 4.0.0Depends on #260.
Test plan
./gradlew :eppo:test./gradlew :eppo:connectedDebugAndroidTest./gradlew :example:installDebug