Skip to content

feat(device_info_plus): Add support of built-in Kotlin#3889

Open
vbuberen wants to merge 2 commits into
mainfrom
feat/device_info_plus_kotlin
Open

feat(device_info_plus): Add support of built-in Kotlin#3889
vbuberen wants to merge 2 commits into
mainfrom
feat/device_info_plus_kotlin

Conversation

@vbuberen

Copy link
Copy Markdown
Collaborator

Description

Backwards compatible support of built-in Kotlin as per https://docs.flutter.dev/release/breaking-changes/migrate-to-built-in-kotlin/for-plugin-authors#supporting-flutter-versions-earlier-than-3-44

Additionally updated example app to not show warnings about versions of Gradle, Kotlin or NDK.

Related Issues

Closes #3835

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
  • No, this is not a breaking change.

@mdmahendri mdmahendri 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.

Just something I caught while testing the PR.

@@ -1,6 +1,5 @@
plugins {
id("com.android.application")
id("kotlin-android")

@mdmahendri mdmahendri Jun 21, 2026

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.

So I just tested this PR locally using Flutter 3.38.1, and the build for the example failed.
The removal of the explicit kotlin-android plugin relies on the auto-apply behavior introduced in Flutter upstream via flutter/flutter#184227.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for catching, but just sharing the link was enough. Such lectures are not needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate Plugin to Built-in Kotlin

2 participants