Skip to content

ci: add minimal remote build verification - #2

Merged
NiDrZero merged 1 commit into
atomux-baseline-cleanupfrom
ci-remote-verification
Jul 24, 2026
Merged

ci: add minimal remote build verification#2
NiDrZero merged 1 commit into
atomux-baseline-cleanupfrom
ci-remote-verification

Conversation

@NiDrZero

Copy link
Copy Markdown
Owner

Summary

Adds the repository's first GitHub Actions workflow to verify compilation and unit tests on a clean remote runner — catching environment issues (especially the JDK version mismatch) that do not surface on a local machine.

Scoped to compile + unit tests, as agreed.

Base

Stacked on top of atomux-baseline-cleanup (PR #1), because the workflow tests the :app module that PR introduces. Merge PR #1 first, then this.

What the workflow does

  • Triggers on pull_request and push against main
  • Pins Temurin JDK 17 — AGP 8.11 supports JDK 17-21, and runners default to newer JDKs that fail. This is the exact issue reproduced locally.
  • Installs Android SDK platform 36 + build-tools 36.0.0 (matches compileSdk/targetSdk = 36)
  • Caches Gradle via gradle/actions/setup-gradle
  • Runs ./gradlew :app:compileDebugKotlin :app:testDebugUnitTest (:app transitively compiles :core)
  • Uploads the unit-test report as an artifact

Deliberately out of scope (v1)

  • APK assembly, :proot-plugin build, and the PowerShell verify-* scripts — easy follow-ups once this baseline is green.

Add a GitHub Actions workflow that compiles :app (transitively :core) and runs
:app unit tests on a clean ubuntu-latest runner.

- Pins Temurin JDK 17 (AGP 8.11 supports 17-21; runner defaults are newer and
  fail otherwise)
- Installs Android SDK platform 36 and build-tools 36.0.0 to match compileSdk
- Caches Gradle and uploads the unit-test report on completion

Triggers on pull_request and push against main.
@NiDrZero
NiDrZero merged commit 1c92b8d into atomux-baseline-cleanup Jul 24, 2026
@NiDrZero
NiDrZero deleted the ci-remote-verification branch July 24, 2026 10:08
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.

1 participant