Skip to content

Conversation

@holzgeist
Copy link
Contributor

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

Document the default-flavor entry in pubspec.yaml introduced in flutter/flutter#147968

Issues fixed by this PR (if any):

Presubmit checklist

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm

@sfshaza2
Copy link
Contributor

/gcbrun

@flutter-website-bot
Copy link
Collaborator

Visit the preview URL for this PR (updated for commit 9b863a6):

https://flutter-docs-prod--pr10632-fix-document-default-flavor-vlw5g0fl.web.app

@sfshaza2 sfshaza2 merged commit 722fa4a into flutter:main May 21, 2024
@holzgeist holzgeist deleted the fix-document-default-flavor branch May 21, 2024 11:39
andrewkolos added a commit that referenced this pull request May 21, 2024
parlough pushed a commit that referenced this pull request May 21, 2024
Reverts #10632. This PR added documentation for a feature
that has not yet landed in stable (nor master, for that matter).
@holzgeist holzgeist mentioned this pull request May 22, 2024
3 tasks
sfshaza2 pushed a commit that referenced this pull request May 22, 2024
_Description of what this PR is changing or adding, and why:_

Document new feature (setting default flavor in pubspec.yaml) introduced
in flutter/flutter#147968

This is a new version of #10632,
please only merge once the PR above is released

_Issues fixed by this PR (if any):_

flutter/flutter#22856

## Presubmit checklist

- [x] This PR doesn’t contain automatically generated corrections
(Grammarly or similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.
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.

3 participants