Skip to content

Migrate addTaskForPrintBuildVariants and isFlutterAppProject to new agp dsl#183849

Merged
auto-submit[bot] merged 3 commits intoflutter:masterfrom
gmackall:small_agp_dsl_migration
Mar 19, 2026
Merged

Migrate addTaskForPrintBuildVariants and isFlutterAppProject to new agp dsl#183849
auto-submit[bot] merged 3 commits intoflutter:masterfrom
gmackall:small_agp_dsl_migration

Conversation

@gmackall
Copy link
Member

@gmackall gmackall commented Mar 18, 2026

Migrate addTaskForPrintBuildVariants and isFlutterAppProject to new agp dsl

@github-actions github-actions bot added platform-android Android applications specifically tool Affects the "flutter" command-line tool. See also t: labels. labels Mar 18, 2026
project.extensions.getByType(ApplicationExtension::class.java)

// Avoid new usages this class is not part of the public AGP DSL.
private fun getAndroidAppExtensionOrNull(project: Project): AbstractAppExtension? =
Copy link
Member Author

Choose a reason for hiding this comment

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

I also deleted this as it was unused

@gmackall gmackall added the CICD Run CI/CD label Mar 18, 2026
Gray Mackall added 2 commits March 18, 2026 11:57
@github-actions github-actions bot removed the CICD Run CI/CD label Mar 18, 2026
@gmackall gmackall marked this pull request as ready for review March 18, 2026 19:00
@gmackall gmackall requested a review from a team as a code owner March 18, 2026 19:00
@gmackall gmackall added the CICD Run CI/CD label Mar 18, 2026
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

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 addTaskForPrintBuildVariants and isFlutterAppProject to use the new AGP DSL, replacing deprecated AbstractAppExtension with ApplicationExtension and AndroidComponentsExtension. The changes involve updating the Gradle plugin and test files to reflect these API changes. The code review focuses on ensuring the correctness of the migration and adherence to the repository's style guide.

@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 8ad8ba0 Mar 19, 2026
152 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 20, 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 20, 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
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.

2 participants