fix: Remove AGP 7.x R8 workaround#1222
Conversation
AGP 7.x is no longer in the CI matrix and the default AGP version is 8.10.1. The conditional R8 classpath override is dead code. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
f5b80a8 to
7fc3951
Compare
romtsn
left a comment
There was a problem hiding this comment.
I guess it's fine to not test compat with 7.x.x -- checked the usage and there's overall around ~500 AGP 7.x.x users (out of those who opted in for telemetry). We should drop it in the next major and it's fine to accept the risk of breaking something for the small number of users for now, imo (it's like 1.5% of overall).
|
I'm seeing this in sentry-java but probably not related to this:
https://github.com/getsentry/sentry-java/actions/runs/28527031334/job/84566138543?pr=5670#step:9:508 |
|
Actually we don't run the release builds against AGP 7.X so this wouldn't affect customers. |
Summary
build.gradle.ktsand the test template inBaseSentryPluginTest.kt#skip-changelog
🤖 Generated with Claude Code