Skip to content

feat: remove kotlin#353

Merged
abelonogov-ld merged 2 commits intomainfrom
andrey/remove-kotlin
Apr 29, 2026
Merged

feat: remove kotlin#353
abelonogov-ld merged 2 commits intomainfrom
andrey/remove-kotlin

Conversation

@abelonogov-ld
Copy link
Copy Markdown
Contributor

@abelonogov-ld abelonogov-ld commented Apr 29, 2026

Requirements

Whole SDK had only one Kotlin file and too high verison of Kotlin was set, 2.2.0 which is above average of the market.
Rewriting one class to Java. It should make Android SDK lighter 1.5MB especially for Pure Java projects


Note

Medium Risk
Build/tooling changes across multiple modules can break CI or consumer builds if any remaining Kotlin usage or plugin assumptions exist. The Java rewrite is straightforward but could subtly change equality/toString behavior for callers.

Overview
Removes Kotlin from the build by dropping the Kotlin Gradle plugin/version configuration, removing org.jetbrains.kotlin.android from modules, and deleting the root KotlinCompile/JvmTarget task configuration.

Replaces the lone Kotlin source (RegistrationCompleteResult.kt) with an equivalent Java implementation (RegistrationCompleteResult.java) to keep the plugin-registration result API available without requiring Kotlin.

Reviewed by Cursor Bugbot for commit 35ca1d5. Bugbot is set up for automated code reviews on this repo. Configure here.

@abelonogov-ld abelonogov-ld requested a review from a team as a code owner April 29, 2026 00:14
Copy link
Copy Markdown
Contributor

@beekld beekld left a comment

Choose a reason for hiding this comment

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

I was on the fence about whether this should count as a breaking change, but I'm thinking it's not. The Kotlin<->Java bridging should paper over the difference.

@abelonogov-ld abelonogov-ld merged commit 5e332a8 into main Apr 29, 2026
6 of 7 checks passed
@abelonogov-ld abelonogov-ld deleted the andrey/remove-kotlin branch April 29, 2026 16:57
beekld pushed a commit that referenced this pull request Apr 29, 2026
🤖 I have created a release *beep* *boop*
---


##
[5.12.0](5.11.2...5.12.0)
(2026-04-29)


### Features

* remove kotlin
([#353](#353))
([5e332a8](5e332a8))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Release-only changes (version bumps and changelog updates) with no
functional code modifications in this diff.
> 
> **Overview**
> Updates the release metadata to publish version **5.12.0** by bumping
the version in `.release-please-manifest.json` and `gradle.properties`.
> 
> Adds a new `CHANGELOG.md` entry for `5.12.0`, noting the feature
"remove kotlin" (per #353).
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
204e908. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants