Skip to content

Only run the assembleRelease target to verify the demo app compiles#1083

Merged
breedx-splk merged 1 commit into
open-telemetry:mainfrom
bidetofevil:demo-app-check
Jul 25, 2025
Merged

Only run the assembleRelease target to verify the demo app compiles#1083
breedx-splk merged 1 commit into
open-telemetry:mainfrom
bidetofevil:demo-app-check

Conversation

@bidetofevil

Copy link
Copy Markdown
Contributor

Reduce the scope of verification of the demo-app by only assembling the release version and bypass debug, lint, and other checks. Doing debug will be even faster but having it go through release and the APK optimizations (including desugaring) is a more comprehensive verification.

@bidetofevil bidetofevil requested a review from a team as a code owner July 22, 2025 17:04

@breedx-splk breedx-splk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given how slow the build is, I'm ok with this, but agree that it's not as thorough. I will wait for @LikeTheSalad to chime in before merging. Thanks!!

@LikeTheSalad

Copy link
Copy Markdown
Contributor

Given how slow the build is, I'm ok with this, but agree that it's not as thorough. I will wait for @LikeTheSalad to chime in before merging. Thanks!!

I think this is fine for the demo app since we mostly just want to avoid breaking its compilation with the agent changes, and usually the most breakable variant is the release one with the R8 stuff that goes on in there 👍

@breedx-splk breedx-splk merged commit d3a3a65 into open-telemetry:main Jul 25, 2025
6 checks passed
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.

3 participants