Skip to content

Update Flutter pubspec page - #13744

Open
sfshaza2 wants to merge 5 commits into
mainfrom
pubspec
Open

Update Flutter pubspec page#13744
sfshaza2 wants to merge 5 commits into
mainfrom
pubspec

Conversation

@sfshaza2

@sfshaza2 sfshaza2 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Fixes #13696

The filed issue requires a minor fix but, in the process, I noticed that the page needs updating.

@sfshaza2
sfshaza2 requested a review from a team as a code owner August 18, 2026 22:17
@sfshaza2
sfshaza2 requested a review from elliette August 18, 2026 22:18

@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 Flutter pubspec documentation to streamline the overview, add information about asset transformers, integration tests, and the module field, and correct various formatting and link references. The feedback recommends using a stable SDK version constraint instead of a beta pre-release in examples, removing the obsolete androidX property from the module configuration, and using shell-style comments (#) instead of C-style comments (//) in console code blocks.

Comment thread sites/docs/src/content/tools/pubspec.md Outdated
Comment thread sites/docs/src/content/tools/pubspec.md Outdated
Comment thread sites/docs/src/content/tools/pubspec.md Outdated
@sfshaza2
sfshaza2 requested a review from justinmc August 18, 2026 22:23
@sfshaza2

Copy link
Copy Markdown
Contributor Author

@elliette @justinmc, with the separation of material/cupertino, is this doc update correct?

@flutter-website-bot

flutter-website-bot commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

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

https://flutter-docs-prod--docs-pr13744-pubspec-ohh0xgww.web.app

@flutter-website-bot

flutter-website-bot commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

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

https://flutter-dev-230821--www-pr13744-pubspec-3hqiahs2.web.app

@elliette elliette left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM but I'll let @justinmc approve since he has more context here.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for removing this!

@justinmc justinmc 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.

LGTM 👍 . I saw nothing contradicting the recent decoupling changes.

Comment thread sites/docs/src/content/tools/pubspec.md Outdated
Comment thread sites/docs/src/content/tools/pubspec.md

@parlough parlough left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @sfshaza2! Some small issues and suggestions.

Also note this PR seems reliant on #13736 landing first.

```

### flutter_localizations package
### flutter_localizations package {: #flutter_localizations }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
### flutter_localizations package {: #flutter_localizations }
### flutter_localizations package {: #flutter-localizations }


environment:
sdk: ^3.12.0
sdk: ^3.14.0-95

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should keep this as a stable version:

Suggested change
sdk: ^3.14.0-95
sdk: ^3.13.0

[Swift Package Manager for plugin authors]: /packages-and-plugins/swift-package-manager/for-plugin-authors

### default-flavor field
### default-flavor field {: #default-flavor }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There's a link to the old header in the deployment/flavors.md page that needs to be updated to the new fragment.

Comment on lines +183 to +185
flavors, visit [Set up flavors for iOS and macOS],
[Set up flavors for Android], and
[Set up flavors for Windows and Linux].

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
flavors, visit [Set up flavors for iOS and macOS],
[Set up flavors for Android], and
[Set up flavors for Windows and Linux].
flavors, visit [Set up flavors for iOS and macOS][],
[Set up flavors for Android][], and
[Set up flavors for Windows and Linux][].


Handles localization tasks. This field can appear as a
subfield of `flutter` and `material`.
Enables generation of localized strings and assets from `.arb` files.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

To avoid potential confusion with the other mention of "assets" on the page:

Suggested change
Enables generation of localized strings and assets from `.arb` files.
Enables generation of Dart files with localized strings from `.arb` files.

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.

"environment" entry is missing

5 participants