Skip to content

build: Move aspectjweaver and kotlin-reflect to the version catalog - #6514

Merged
runningcode merged 1 commit into
mainfrom
no/catalog-aspectj-kotlin-reflect
Nov 9, 2026
Merged

runningcode merged 1 commit into
mainfrom
no/catalog-aspectj-kotlin-reflect

Conversation

@runningcode

@runningcode runningcode commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

📜 Description

Move aspectjweaver and kotlin-reflect to gradle/libs.versions.toml version catalog.

One version change: sentry-apollo-4 pinned kotlin-reflect:2.0.0 in its tests, but mockito-kotlin already pulled in 2.1.20, so the pin had no effect. It now resolves to 2.3.21, the same version as stdlib.

💡 Motivation and Context

Keep dependency coordinates in one place, the version catalog.

💚 How did you test it?

Checked with dependencyInsight that the root and samples builds resolve the same versions as before, except in the sentry-apollo-4 tests, which pass with 2.3.21.

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

🤖 Generated with Claude Code

#skip-changelog

Replace the string coordinates and the Config.Libs constants with catalog entries. Both stay versionless: Spring BOMs manage aspectjweaver and the Kotlin plugin manages kotlin-reflect.

sentry-apollo-4 pinned kotlin-reflect to 2.0.0 in tests, but mockito-kotlin already pulled in 2.1.20, so the pin had no effect. It now resolves to 2.3.21, the same version as stdlib.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@runningcode runningcode added the ship-it PR is ready to merge from a reviewer perspective label Nov 9, 2026
@runningcode
runningcode marked this pull request as ready for review November 9, 2026 08:07
@runningcode
runningcode requested review from a team and adinauer as code owners November 9, 2026 08:07
@runningcode
runningcode requested review from 0xadam-brown and markushi and removed request for a team November 9, 2026 08:07
@sentry

sentry Bot commented Nov 9, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.63.0 (1) release

⚙️ sentry-android Build Distribution Settings

@runningcode
runningcode merged commit 4a2c19f into main Nov 9, 2026
48 of 51 checks passed
@runningcode
runningcode deleted the no/catalog-aspectj-kotlin-reflect branch November 9, 2026 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ship-it PR is ready to merge from a reviewer perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants