Visitar URL original
Document environment-conditioned assets by Calpoog · Pull Request #13886 · flutter/website · GitHub
Skip to content

Document environment-conditioned assets - #13886

Draft
Calpoog wants to merge 1 commit into
flutter:mainfrom
Calpoog:environment-conditioned-assets-docs
Draft

Calpoog wants to merge 1 commit into
flutter:mainfrom
Calpoog:environment-conditioned-assets-docs

Conversation

@Calpoog

@Calpoog Calpoog commented Sep 14, 2026

Copy link
Copy Markdown

Description of what this PR is changing or adding, and why:

Documents the proposed environment subfield for Flutter asset declarations. The assets guide explains exact matching, AND semantics across variables, OR semantics for value lists, unset-variable behavior, and composition with flavor and platform conditions. The pubspec reference defines the field and includes scalar and list examples.

Issues fixed by this PR (if any):

Supports flutter/flutter#166842.

PRs or commits this PR depends on (if any):

Blocked by flutter/flutter#192739. This documentation should remain draft until the framework feature is accepted and its release availability is known.

Validation

  • dart run dash_site --site=docs build
  • dart run dash_site --site=docs check-link-references
  • dart run dash_site --site=docs check-links

check-all also completed tests and excerpt validation, but its analyzer process crashed while attempting to send analytics because the network connection was cancelled. Its format check reports three pre-existing unrelated Dart files.

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.

@google-cla

google-cla Bot commented Sep 14, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@Calpoog
Calpoog force-pushed the environment-conditioned-assets-docs branch from dcea1d2 to cca0a19 Compare September 14, 2026 17:52
@sfshaza2

Copy link
Copy Markdown
Contributor

/gbcrun

@sfshaza2

Copy link
Copy Markdown
Contributor

@Calpoog, I know this is still in draft form, but I wanted to run the build on it, to see how it goes.

@Calpoog

Calpoog commented Sep 21, 2026

Copy link
Copy Markdown
Author

Hi @sfshaza2,

I've kept this one in draft until the implementation side is complete. However, I've still got that one in draft. Is there a particular draft/non-draft order I should doing these PRs in? This is my first time, so not sure of the process :)

@sfshaza2

Copy link
Copy Markdown
Contributor

No, this is fine, @Calpoog. I just wanted to see if the build/bot was copacetic, and it seems to be. :)

@Calpoog

Calpoog commented Sep 22, 2026

Copy link
Copy Markdown
Author

@sfshaza2 should I take the flutter repo implementation PR out of draft? No activity over there for a bit.

@sfshaza2

Copy link
Copy Markdown
Contributor

Hi @Calpoog! I went ahead and ran the tests on that PR. The thing is, reviewers often ignore a PR until it's out of draft mode. So, I leave that up to you. :D

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