Parsely / parsely-android

The official Parse.ly Android toolkit -- for instrumenting your apps with Parse.ly
https://docs.parse.ly/android-sdk/
Other
6 stars 6 forks source link

Migrate threading model to Coroutines - long running branch #89

Closed wzieba closed 11 months ago

wzieba commented 1 year ago

Description

This PR groups PRs migrated the SDK from AsyncTask/java.utils.Timer APIs to Kotlin Coroutines. The migration has a few goals in mind

Contains changes from:

codecov[bot] commented 1 year ago

Codecov Report

Attention: 24 lines in your changes are missing coverage. Please review.

Comparison is base (1e6e189) 51.22% compared to head (ec44c99) 68.24%.

Files Patch % Lines
...ava/com/parsely/parselyandroid/ParselyTracker.java 60.00% 7 Missing and 1 partial :warning:
...va/com/parsely/parselyandroid/EngagementManager.kt 82.85% 3 Missing and 3 partials :warning:
.../parsely/parselyandroid/AdvertisementIdProvider.kt 84.21% 3 Missing :warning:
.../java/com/parsely/parselyandroid/JsonSerializer.kt 75.00% 2 Missing and 1 partial :warning:
...in/java/com/parsely/parselyandroid/FlushManager.kt 81.81% 1 Missing and 1 partial :warning:
.../java/com/parsely/parselyandroid/InMemoryBuffer.kt 95.65% 0 Missing and 1 partial :warning:
...com/parsely/parselyandroid/ParselyAPIConnection.kt 90.90% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #89 +/- ## =========================================== + Coverage 51.22% 68.24% +17.01% =========================================== Files 10 16 +6 Lines 367 359 -8 Branches 56 52 -4 =========================================== + Hits 188 245 +57 + Misses 165 97 -68 - Partials 14 17 +3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.