[various] Convert plugin builds to Kotlin gradle#11172
[various] Convert plugin builds to Kotlin gradle#11172auto-submit[bot] merged 37 commits intoflutter:mainfrom
Conversation
There was a problem hiding this comment.
Code Review
This pull request migrates a large number of Android plugin build files from Groovy to Kotlin DSL. The changes are mostly mechanical conversions and appear to be correct, preserving the original build logic. I've found one minor issue where a compiler option was dropped during the conversion in one of the test plugins, which I've commented on.
| kotlinOptions { | ||
| // This must match the Java version provided in compileOptions. | ||
| jvmTarget = JavaVersion.VERSION_17.toString() | ||
| } |
There was a problem hiding this comment.
The kotlin { compilerOptions { ... } } section on line 31 is the newer form of this.
| // The following Espresso dependency can be either "implementation" | ||
| // or "androidTestImplementation", depending on whether you want the | ||
| // dependency to appear on your APK's compile classpath or the test APK | ||
| // dependency to appear on your APK"s compile classpath or the test APK |
There was a problem hiding this comment.
this doesn't seem intentional
There was a problem hiding this comment.
Oops, I thought I fixed all of these. My vi record/playback for classpath standardization wasn't quite as precise as intended on the initial pass.
| kotlinOptions { | ||
| jvmTarget = JavaVersion.VERSION_17.toString() | ||
| } | ||
|
|
There was a problem hiding this comment.
here also, I saw that it was still in some of the new files
There was a problem hiding this comment.
Same. The new files should all have the new form rather than this.
flutter/packages@8dcfd11...5909bdd 2026-03-25 47866232+chunhtai@users.noreply.github.com [ci] add more permissions for create-pull-request (flutter/packages#11302) 2026-03-25 stuartmorgan@google.com [various] Add `unintended_html_in_doc_comment` to analysis options (flutter/packages#11303) 2026-03-25 matt.boetger@gmail.com Use deprecated dependency until legacy renderer is removed (flutter/packages#11185) 2026-03-25 louisehsu@google.com [in_app_purchase_storekit] Address flaky tests (flutter/packages#11270) 2026-03-25 stuartmorgan@google.com [google_maps_flutter] Fix A2A iOS builds (flutter/packages#11290) 2026-03-25 stuartmorgan@google.com [mustache_template] Fix broken README link (flutter/packages#11306) 2026-03-25 stuartmorgan@google.com [ci] Add a workflow to auto-remove CICD label (flutter/packages#11301) 2026-03-25 stuartmorgan@google.com [various] Convert plugin builds to Kotlin gradle (flutter/packages#11172) 2026-03-25 spkhalad@gmail.com [webview_flutter_platform_interface] Add support for getting cookie (flutter/packages#11037) 2026-03-25 stuartmorgan@google.com [cupertino_icons] Remove empty Dart file (flutter/packages#11308) 2026-03-25 stuartmorgan@google.com [camera] Regenerate iOS example with Swift (flutter/packages#11283) 2026-03-25 tarrinneal@gmail.com [pigeon] Optimize data class equality and hashing in Dart, Kotlin, java, and Swift, adds equality in other languages (flutter/packages#11140) 2026-03-25 katelovett@google.com [two_dimensional_scrollables] Fix span border decorations in flipped cross axes (flutter/packages#11334) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC flutter-ecosystem@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
As a precursor to converting plugins to use Kotlin Gradle build files in flutter#11172, this does some final standardization of the Groovy versions: - Uses `"` instead of `'` on more lines. - Uses `(...)` and `=` in places where it's optional, instead of just a space. - Uses `minSdk` instead of `minSdkVersion`. - Standardizes vertical whitespace (not necessary for the conversion, but it's something I cleaned up in that PR). This is being done as a precursor PR because I noticed that the files are mostly right on the cusp of being similar enough for git to think they are renames with changes rather than deletion and addition; some are ending up one way, and some the other. Since renames with changes make blame much more useful, I'd like to make the files more similar so that hopefully almost all of them will end up as renames with changes in that PR. ## Pre-Review Checklist [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
Following up from flutter#11169, this converts all the rest of the plugins in the repository to use Kotlin rather than Groovy for plugin build files. As with that PR, this does not change the example apps, only the plugin builds themselves. Part of flutter/flutter#176065 ## Pre-Review Checklist [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
…er#184123) flutter/packages@8dcfd11...5909bdd 2026-03-25 47866232+chunhtai@users.noreply.github.com [ci] add more permissions for create-pull-request (flutter/packages#11302) 2026-03-25 stuartmorgan@google.com [various] Add `unintended_html_in_doc_comment` to analysis options (flutter/packages#11303) 2026-03-25 matt.boetger@gmail.com Use deprecated dependency until legacy renderer is removed (flutter/packages#11185) 2026-03-25 louisehsu@google.com [in_app_purchase_storekit] Address flaky tests (flutter/packages#11270) 2026-03-25 stuartmorgan@google.com [google_maps_flutter] Fix A2A iOS builds (flutter/packages#11290) 2026-03-25 stuartmorgan@google.com [mustache_template] Fix broken README link (flutter/packages#11306) 2026-03-25 stuartmorgan@google.com [ci] Add a workflow to auto-remove CICD label (flutter/packages#11301) 2026-03-25 stuartmorgan@google.com [various] Convert plugin builds to Kotlin gradle (flutter/packages#11172) 2026-03-25 spkhalad@gmail.com [webview_flutter_platform_interface] Add support for getting cookie (flutter/packages#11037) 2026-03-25 stuartmorgan@google.com [cupertino_icons] Remove empty Dart file (flutter/packages#11308) 2026-03-25 stuartmorgan@google.com [camera] Regenerate iOS example with Swift (flutter/packages#11283) 2026-03-25 tarrinneal@gmail.com [pigeon] Optimize data class equality and hashing in Dart, Kotlin, java, and Swift, adds equality in other languages (flutter/packages#11140) 2026-03-25 katelovett@google.com [two_dimensional_scrollables] Fix span border decorations in flipped cross axes (flutter/packages#11334) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC flutter-ecosystem@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
As a precursor to converting plugins to use Kotlin Gradle build files in flutter/packages#11172, this does some final standardization of the Groovy versions: - Uses `"` instead of `'` on more lines. - Uses `(...)` and `=` in places where it's optional, instead of just a space. - Uses `minSdk` instead of `minSdkVersion`. - Standardizes vertical whitespace (not necessary for the conversion, but it's something I cleaned up in that PR). This is being done as a precursor PR because I noticed that the files are mostly right on the cusp of being similar enough for git to think they are renames with changes rather than deletion and addition; some are ending up one way, and some the other. Since renames with changes make blame much more useful, I'd like to make the files more similar so that hopefully almost all of them will end up as renames with changes in that PR. ## Pre-Review Checklist [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
…er#184123) flutter/packages@8dcfd11...5909bdd 2026-03-25 47866232+chunhtai@users.noreply.github.com [ci] add more permissions for create-pull-request (flutter/packages#11302) 2026-03-25 stuartmorgan@google.com [various] Add `unintended_html_in_doc_comment` to analysis options (flutter/packages#11303) 2026-03-25 matt.boetger@gmail.com Use deprecated dependency until legacy renderer is removed (flutter/packages#11185) 2026-03-25 louisehsu@google.com [in_app_purchase_storekit] Address flaky tests (flutter/packages#11270) 2026-03-25 stuartmorgan@google.com [google_maps_flutter] Fix A2A iOS builds (flutter/packages#11290) 2026-03-25 stuartmorgan@google.com [mustache_template] Fix broken README link (flutter/packages#11306) 2026-03-25 stuartmorgan@google.com [ci] Add a workflow to auto-remove CICD label (flutter/packages#11301) 2026-03-25 stuartmorgan@google.com [various] Convert plugin builds to Kotlin gradle (flutter/packages#11172) 2026-03-25 spkhalad@gmail.com [webview_flutter_platform_interface] Add support for getting cookie (flutter/packages#11037) 2026-03-25 stuartmorgan@google.com [cupertino_icons] Remove empty Dart file (flutter/packages#11308) 2026-03-25 stuartmorgan@google.com [camera] Regenerate iOS example with Swift (flutter/packages#11283) 2026-03-25 tarrinneal@gmail.com [pigeon] Optimize data class equality and hashing in Dart, Kotlin, java, and Swift, adds equality in other languages (flutter/packages#11140) 2026-03-25 katelovett@google.com [two_dimensional_scrollables] Fix span border decorations in flipped cross axes (flutter/packages#11334) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC flutter-ecosystem@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Following up from #11169, this converts all the rest of the plugins in the repository to use Kotlin rather than Groovy for plugin build files. As with that PR, this does not change the example apps, only the plugin builds themselves.
Part of flutter/flutter#176065
Pre-Review Checklist
[shared_preferences]///).Footnotes
Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. ↩ ↩2