[various] Plugin Gradle pre-Kotlin standardization#11173
Conversation
|
Version/changelog override: I'm intentionally not releasing these since the changes are no-ops in practice, and we'll be releasing all of these changes almost immediate afteward as part of #11172. |
|
@gmackall This just needs a quick rubber stamp to streamline the next PR in the conversion series. |
There was a problem hiding this comment.
Code Review
This pull request performs a broad standardization of Groovy Gradle build files across numerous packages. The changes apply more modern Gradle DSL syntax, such as using double quotes, parentheses for method calls, the = for assignments, and updating properties like minSdkVersion to minSdk and lintOptions to lint, in preparation for a migration to Kotlin Gradle files.
The standardization appears to be incomplete for packages/local_auth/local_auth_android/android/build.gradle. This file still contains older syntax (e.g., single quotes for group and version, and method calls without parentheses) that has been updated in other files in this PR. Applying the standardization consistently to all files would better achieve the PR's stated goal.
local_auth wasn't supposed to be included, to avoid conflicts with #11169. Reverted. |
flutter/packages@fe3de64...c717018 2026-03-09 23180853+jeffkwoh@users.noreply.github.com [pigeon] Support javax.annotation.Generated annotation in Kotlin generator (flutter/packages#10961) 2026-03-09 49699333+dependabot[bot]@users.noreply.github.com [dependabot]: Bump gradle-wrapper from 8.13 to 9.4.0 in /packages/path_provider/path_provider/example/android/app (flutter/packages#11206) 2026-03-09 49699333+dependabot[bot]@users.noreply.github.com [dependabot]: Bump com.android.tools.build:gradle from 8.9.1 to 9.1.0 in /packages/shared_preferences/shared_preferences_android/example/android/app (flutter/packages#11208) 2026-03-09 49699333+dependabot[bot]@users.noreply.github.com [dependabot]: Bump gradle-wrapper from 8.13 to 9.4.0 in /packages/path_provider/path_provider_android/example/android/app (flutter/packages#11207) 2026-03-06 stuartmorgan@google.com [various] Plugin Gradle pre-Kotlin standardization (flutter/packages#11173) 2026-03-06 jessiewong401@gmail.com Updated AGP and KGP to align with flutter/flutter templates (flutter/packages#10423) 2026-03-06 engine-flutter-autoroll@skia.org Roll Flutter from d3dd774 to d182143 (33 revisions) (flutter/packages#11191) 2026-03-06 20989940+aednlaxer@users.noreply.github.com [google_maps_flutter] Add Advanced Markers support (flutter/packages#7882) 2026-03-06 47250617+linchen00@users.noreply.github.com [in_app_purchase]Fixes StoreKit 2 purchase flow to send cancelled/pending/unverified results to purchaseStream. (flutter/packages#10736) 2026-03-06 theandroidseb@gmail.com [flutter_svg] Update README with an example to scale images (flutter/packages#10547) 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
…er#183396) flutter/packages@fe3de64...c717018 2026-03-09 23180853+jeffkwoh@users.noreply.github.com [pigeon] Support javax.annotation.Generated annotation in Kotlin generator (flutter/packages#10961) 2026-03-09 49699333+dependabot[bot]@users.noreply.github.com [dependabot]: Bump gradle-wrapper from 8.13 to 9.4.0 in /packages/path_provider/path_provider/example/android/app (flutter/packages#11206) 2026-03-09 49699333+dependabot[bot]@users.noreply.github.com [dependabot]: Bump com.android.tools.build:gradle from 8.9.1 to 9.1.0 in /packages/shared_preferences/shared_preferences_android/example/android/app (flutter/packages#11208) 2026-03-09 49699333+dependabot[bot]@users.noreply.github.com [dependabot]: Bump gradle-wrapper from 8.13 to 9.4.0 in /packages/path_provider/path_provider_android/example/android/app (flutter/packages#11207) 2026-03-06 stuartmorgan@google.com [various] Plugin Gradle pre-Kotlin standardization (flutter/packages#11173) 2026-03-06 jessiewong401@gmail.com Updated AGP and KGP to align with flutter/flutter templates (flutter/packages#10423) 2026-03-06 engine-flutter-autoroll@skia.org Roll Flutter from d3dd774 to d182143 (33 revisions) (flutter/packages#11191) 2026-03-06 20989940+aednlaxer@users.noreply.github.com [google_maps_flutter] Add Advanced Markers support (flutter/packages#7882) 2026-03-06 47250617+linchen00@users.noreply.github.com [in_app_purchase]Fixes StoreKit 2 purchase flow to send cancelled/pending/unverified results to purchaseStream. (flutter/packages#10736) 2026-03-06 theandroidseb@gmail.com [flutter_svg] Update README with an example to scale images (flutter/packages#10547) 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 #11172, this does some final standardization of the Groovy versions:
"instead of'on more lines.(...)and=in places where it's optional, instead of just a space.minSdkinstead ofminSdkVersion.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
[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