Skip to content

feat(audience): drop Mono from CI matrix and fix Android build size#805

Merged
nattb8 merged 2 commits into
mainfrom
feat/audience-android-size-drop-mono
Jun 28, 2026
Merged

feat(audience): drop Mono from CI matrix and fix Android build size#805
nattb8 merged 2 commits into
mainfrom
feat/audience-android-size-drop-mono

Conversation

@nattb8

@nattb8 nattb8 commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator
  • Removes Mono from the scripting backend matrix (SDK-548); IL2CPP is stricter and covers mobile, so Mono adds cost without meaningful coverage.
  • Fixes Android build size always reporting 0.00 MB (SDK-540 follow-up) by forcing exportAsGoogleAndroidProject = false in AndroidBuilder, so Unity outputs a direct APK rather than a Gradle project directory that the size check couldn't find.

🤖 Generated with Claude Code

@nattb8 nattb8 requested review from a team as code owners June 26, 2026 07:38
IL2CPP covers the cases that matter (mobile is IL2CPP-only); Mono was doubling
the desktop test count for negligible extra coverage.

Adds exportAsGoogleAndroidProject = false to AndroidBuilder so the build always
produces a direct APK rather than a Gradle project directory, which was causing
the build-size check to report 0.00 MB for Android.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nattb8 nattb8 force-pushed the feat/audience-android-size-drop-mono branch from 84dfbcf to 056dff8 Compare June 26, 2026 07:43
@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown

Audience SDK — Build Size

Platform Build Size vs Baseline
Android 21.61 MB +21.61 MB
Windows 51.38 MB +51.38 MB
iOS 47.29 MB +47.29 MB
macOS 91.86 MB +91.86 MB

No size limits set yet. Set baselineBytes and maxBytes in .github/audience-build-budget.json.

apkSize() only scanned the top level of the downloaded artifact directory.
The combined upload path (Android + Windows globs) causes upload-artifact to
set the common ancestor to Builds/, so the APK lands at
artifacts/Android/Android/audience.apk — one level deeper than apkSize expected.

dirSize() already walks recursively so it finds the APK regardless of nesting,
and the Android artifact only ever contains the APK so the result is the same.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nattb8 nattb8 merged commit 7642e40 into main Jun 28, 2026
61 of 70 checks passed
@nattb8 nattb8 deleted the feat/audience-android-size-drop-mono branch June 28, 2026 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants