Skip to content

docs: update android buildTypes example in flavors guide - #13674

Open
lamek wants to merge 1 commit into
mainfrom
fix-12205
Open

docs: update android buildTypes example in flavors guide#13674
lamek wants to merge 1 commit into
mainfrom
fix-12205

Conversation

@lamek

@lamek lamek commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Updated the build.gradle.kts snippet to reflect that the default Flutter template only explicitly configures the release build type, leaving debug as implicit

#12205

Presubmit checklist

  • If you are unwilling, or unable, to sign the CLA, even for a tiny, one-word PR, please file an issue instead of a PR.
  • If this PR is not meant to land until a future stable release, mark it as draft with an explanation.
  • This PR follows the Google Developer Documentation Style Guidelines—for example, it doesn't use i.e. or e.g., and it avoids I and we (first-person pronouns).
  • This PR uses semantic line breaks
    of 80 characters or fewer.

@lamek
lamek requested review from a team and sfshaza2 as code owners August 4, 2026 20:15

@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 updates the documentation in flavors.md regarding the configuration of build types in build.gradle.kts for Flutter. The review feedback correctly points out that using release { ... } directly in Gradle Kotlin DSL can cause compilation errors, and suggests using getByName("release") instead to ensure compatibility and match the default Flutter template.

Comment thread sites/docs/src/content/deployment/flavors.md
@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit 94827a0):

https://flutter-docs-prod--docs-pr13674-fix-12205-655f5goj.web.app

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit 94827a0):

https://flutter-dev-230821--www-pr13674-fix-12205-n2m5wrqe.web.app

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants