Add sentry-android-distribution module to release configuration #4791
Conversation
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Instructions and example for changelogPlease add an entry to Example: ## Unreleased
- Add sentry-android-distribution module to release configuration ([#4791](https://github.com/getsentry/sentry-java/pull/4791))If none of the above apply, you can opt out of this check by adding |
| } | ||
|
|
||
| if (!this.name.contains("sample") && !this.name.contains("integration-tests") && this.name != "sentry-system-test-support" && this.name != "sentry-test-support" && this.name != "sentry-android-distribution") { | ||
| if (!this.name.contains("sample") && !this.name.contains("integration-tests") && this.name != "sentry-system-test-support" && this.name != "sentry-test-support") { |
There was a problem hiding this comment.
Potential bug: Incomplete distribution module will crash when used
-
Description: Publishing the
sentry-android-distributionmodule exposes an incomplete implementation that throws aNotImplementedErrorwhen thecheckForUpdateBlockingmethod is called, which will cause a crash for users. -
Suggested fix: Complete the implementation of the
checkForUpdateBlockingmethod inDistributionIntegration.ktto handle update checks gracefully instead of throwing aNotImplementedError. Alternatively, if the module is not ready for public use, revert the change that publishes it to Maven Central.
severity: 3.0, confidence: 5.0
Did we get this right? 👍 / 👎 to inform future reviews.
| maven:io.sentry:sentry-apache-http-client-5: | ||
| maven:io.sentry:sentry-android: | ||
| maven:io.sentry:sentry-android-core: | ||
| maven:io.sentry:sentry-android-distribution: |
There was a problem hiding this comment.
I think for the first time it has to be manually added as described here: https://github.com/getsentry/sentry-release-registry#adding-new-sdks
After that we can merge the PR and it will be automatically published to the registry for future releases
There was a problem hiding this comment.
Oh. I was able to make a release already here: https://repo1.maven.org/maven2/io/sentry/sentry-android-distribution/8.24.0-alpha.1/
but I will do this!
Summary
This ensures the sentry-android-distribution module follows all requirements from
.cursor/rules/new_module.mdcand will be properly published to Maven Central.Solves EME-399
🤖 Generated with Claude Code