Repository navigation
Update docs and links for decoupling - #191231
Conversation
| - `animations` | ||
| - `cupertino_icons` | ||
| - `flutter_lints` | ||
| - `go_router`, `go_router_builder` | ||
| - `google_fonts` | ||
| - `two_dimensional_scrollables` | ||
| - `vector_math` |
There was a problem hiding this comment.
We'll sort out the rest of these @justinmc just figuring out what triage meeting they fit into now. :)
There was a problem hiding this comment.
Code Review
This pull request updates the repository configuration and documentation to reflect the migration of the Material and Cupertino design libraries to separate packages (material_ui and cupertino_ui) with updated labels (p: material_ui and p: cupertino_ui). The feedback suggests correcting GitHub directory links in Project-teams.md to use /tree/main/ instead of /blob/main/ and removing a trailing + from a query URL in README.md.
| * The material library ([flutter/packages packages/material_ui](https://github.com/flutter/packages/blob/main/packages/material_ui); label ["p: material_ui"](https://github.com/flutter/flutter/labels/p%3A%20material_ui)) | ||
|
|
||
| * The cupertino library ([flutter/flutter packages/flutter/lib/src/cupertino](https://github.com/flutter/flutter/blob/main/packages/flutter/lib/src/cupertino); label ["f: cupertino"](https://github.com/flutter/flutter/labels/f%3A%20cupertino)) | ||
| * The cupertino library ([flutter/packages packages/cupertino_ui](https://github.com/flutter/packages/blob/main/packages/cupertino_ui); label ["p: cupertino_ui"](https://github.com/flutter/flutter/labels/p%3A%20cupertino_ui)) |
There was a problem hiding this comment.
Since packages/material_ui and packages/cupertino_ui are directories in the flutter/packages repository, the URLs should use /tree/main/ instead of /blob/main/ to correctly link to them on GitHub.
| * The material library ([flutter/packages packages/material_ui](https://github.com/flutter/packages/blob/main/packages/material_ui); label ["p: material_ui"](https://github.com/flutter/flutter/labels/p%3A%20material_ui)) | |
| * The cupertino library ([flutter/flutter packages/flutter/lib/src/cupertino](https://github.com/flutter/flutter/blob/main/packages/flutter/lib/src/cupertino); label ["f: cupertino"](https://github.com/flutter/flutter/labels/f%3A%20cupertino)) | |
| * The cupertino library ([flutter/packages packages/cupertino_ui](https://github.com/flutter/packages/blob/main/packages/cupertino_ui); label ["p: cupertino_ui"](https://github.com/flutter/flutter/labels/p%3A%20cupertino_ui)) | |
| * The material library ([flutter/packages packages/material_ui](https://github.com/flutter/packages/tree/main/packages/material_ui); label ["p: material_ui"](https://github.com/flutter/flutter/labels/p%3A%20material_ui)) | |
| * The cupertino library ([flutter/packages packages/cupertino_ui](https://github.com/flutter/packages/tree/main/packages/cupertino_ui); label ["p: cupertino_ui"](https://github.com/flutter/flutter/labels/p%3A%20cupertino_ui)) |
| - [Legacy Design Languages PRs](https://github.com/flutter/flutter/pulls?q=is%3Aopen+is%3Apr+label%3A%22f%3A+material+design%22%2C%22f%3A+cupertino%22+sort%3Aupdated-asc+draft%3Afalse) | ||
| - [Design Language PRs](https://github.com/flutter/packages/pulls?q=sort%3Aupdated-desc+is%3Apr+is%3Aopen+label%3A%22p%3A+material_ui%22%2C%22p%3A+cupertino_ui%22+draft%3Afalse) | ||
| - [Legacy Design Languages PRs](https://github.com/flutter/flutter/pulls?q=is%3Aopen+is%3Apr+label%3A%22f%3A+material+design%22%2C%22f%3A+cupertino%22%2C%22p%3A+cupertino_ui%22%2C%22p%3A+material_ui%22+sort%3Aupdated-asc+draft%3Afalse) | ||
| - [Design Language PRs](https://github.com/flutter/packages/pulls?q=sort%3Aupdated-desc+is%3Apr+is%3Aopen+label%3A%22p%3A+material_ui%22%2C%22p%3A+cupertino_ui%22%2C%22triage-design%22+draft%3Afalse+) |
There was a problem hiding this comment.
There is an extra trailing + at the end of the URL query string (draft%3Afalse+). Removing it keeps the URL clean and prevents any potential query parsing issues.
| - [Design Language PRs](https://github.com/flutter/packages/pulls?q=sort%3Aupdated-desc+is%3Apr+is%3Aopen+label%3A%22p%3A+material_ui%22%2C%22p%3A+cupertino_ui%22%2C%22triage-design%22+draft%3Afalse+) | |
| - [Design Language PRs](https://github.com/flutter/packages/pulls?q=sort%3Aupdated-desc+is%3Apr+is%3Aopen+label%3A%22p%3A+material_ui%22%2C%22p%3A+cupertino_ui%22%2C%22triage-design%22+draft%3Afalse) |
Piinks
left a comment
There was a problem hiding this comment.
Just FYI, I kept the old f: material design and f: cupertino in the search strings for now. It will take a while for GitHub to reflect the bulk update to p: material_ui and p: cupertino_ui
Updates the triage docs and the auto-labeler for the post-decoupling world.
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.
If this change needs to override an active code freeze, provide a comment explaining why. The code freeze workflow can be overridden by code reviewers. See pinned issues for any active code freezes with guidance.
Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the
gemini-code-assistbot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.