Idrinth / habit-evaluator / 21922596047

40%

push

github

Speed up Android CI by eliminating redundant R8 builds (#338)

* Speed up Android CI by eliminating redundant R8 builds

The ./gradlew build in the java job was building all 4 Android variants
(modern/legacy × debug/release) including 2 expensive R8 minification
passes, lint for all variants, and JaCoCo-instrumented tests — none of
which were used by that job. Combined with 4 separate apk/aab jobs each
doing their own R8 pass on fresh runners, the pipeline ran 6 total R8
passes.

Changes:
- Scope java job to :shared:build :webserver:build :desktop:build only
  (removes unused Android + website/homepage builds from critical path)
- Add dedicated android job that builds debug variants and runs tests in
  parallel with the java job (no R8, no lint, no release variants)
- Merge separate apk and aab jobs into single android-release job per
  flavor, building APK + AAB in one Gradle invocation to share the R8
  pass (6 R8 passes → 2)
- Apply same apk/aab merge to release.yml (4 R8 passes → 2)

https://claude.ai/code/session_012iitmkGkTN3Z1bc8cDJTHp

* Add Android coverage to Coveralls aggregation

Add a Coveralls upload step to the android CI job pointing at the
JaCoCo XML report, and include the android job in the coveralls-finish
needs list and carryforward flags so android coverage is aggregated
with shared, webserver, desktop, and website.

https://claude.ai/code/session_012iitmkGkTN3Z1bc8cDJTHp

* Scope jacocoTestReport to built modules in java CI job

The unscoped ./gradlew jacocoTestReport was causing Gradle to resolve
:android:jacocoTestReport (and its test task dependencies) even though
the android module is not built in this job. Scope the invocation to
:shared, :webserver, and :desktop only. jacocoAggregateReport and
jacocoTestCoverageVerification already scope to javaModules in the
root build.gradle so they need no change.

https://claude.ai/code/session_012iitmkGkTN3Z1bc8cDJTHp

---------

Co-authored-by: Claude <noreply@anthropic.com>

44 of 69 branches covered (63.77%)

Branch coverage included in aggregate %.

101 of 141 relevant lines covered (71.63%)

11.23 hits per line