Skip to content

Migrate profile build type creation and proguard config to new dsl#183856

Merged
auto-submit[bot] merged 5 commits into
flutter:masterfrom
gmackall:more_more_agp_dsl_migration
Mar 19, 2026
Merged

Migrate profile build type creation and proguard config to new dsl#183856
auto-submit[bot] merged 5 commits into
flutter:masterfrom
gmackall:more_more_agp_dsl_migration

Conversation

@gmackall

@gmackall gmackall commented Mar 18, 2026

Copy link
Copy Markdown
Member

Migrate profile build type creation and proguard config to new dsl

part of #181557

@gmackall gmackall marked this pull request as ready for review March 18, 2026 19:40
@gmackall gmackall requested a review from a team as a code owner March 18, 2026 19:40
@gmackall gmackall added the CICD Run CI/CD label Mar 18, 2026
@github-actions github-actions Bot added platform-android Android applications specifically tool Affects the "flutter" command-line tool. See also t: labels. and removed CICD Run CI/CD labels Mar 18, 2026

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request migrates the creation of the profile build type and Proguard configuration to use the newer Android Gradle Plugin DSL. The changes are mostly correct, but I've identified a regression where custom Proguard rules might not be applied, and an opportunity to improve code readability by reducing redundant function calls. My review includes a suggestion to address both issues.

Comment thread packages/flutter_tools/gradle/src/main/kotlin/FlutterPlugin.kt
@gmackall gmackall added the CICD Run CI/CD label Mar 18, 2026
reidbaker
reidbaker previously approved these changes Mar 19, 2026
@github-actions github-actions Bot removed the CICD Run CI/CD label Mar 19, 2026
@gmackall gmackall added the CICD Run CI/CD label Mar 19, 2026
@gmackall gmackall requested a review from reidbaker March 19, 2026 21:15
@gmackall gmackall added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 19, 2026
@auto-submit

auto-submit Bot commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/183856, because - The status or check suite Mac_x64 framework_tests_misc has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 19, 2026
@gmackall gmackall added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 19, 2026
@auto-submit auto-submit Bot added this pull request to the merge queue Mar 19, 2026
Merged via the queue into flutter:master with commit e95c461 Mar 19, 2026
155 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 19, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 20, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 21, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 21, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 22, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 22, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 23, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 23, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 23, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 23, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 24, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 24, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 24, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 24, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 24, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 24, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 25, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 25, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 25, 2026
mboetger pushed a commit to mboetger/flutter that referenced this pull request Mar 26, 2026
…lutter#183856)

Migrate profile build type creation and proguard config to new dsl

---------

Co-authored-by: Gray Mackall <mackall@google.com>
ahmedsameha1 pushed a commit to ahmedsameha1/flutter that referenced this pull request Apr 14, 2026
…lutter#183856)

Migrate profile build type creation and proguard config to new dsl

---------

Co-authored-by: Gray Mackall <mackall@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD platform-android Android applications specifically tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants