Skip to content

[material_ui] Add a README and placeholders for migrating gen_defaults templates - #12220

Merged
elliette merged 6 commits into
flutter:m3e_migrationfrom
elliette:gen-defaults-migration-readme
Jul 16, 2026
Merged

elliette merged 6 commits into
flutter:m3e_migrationfrom
elliette:gen-defaults-migration-readme

Conversation

@elliette

@elliette elliette commented Jul 16, 2026 •

Copy link
Copy Markdown
Member

Work towards flutter/flutter#187899

  • Adds a README on how to migrate the gen_defaults templates, to be used by contributors or agents.
  • Adds placeholders (referenced in README) for templates to prevent merge conflicts while migrating.

Pre-Review Checklist

Test exemption: Markdown file only.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot 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.

Footnotes

  1. Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. ↩ ↩2

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Jul 16, 2026
@elliette
elliette requested a review from QuncCccccc July 16, 2026 19:52
@github-actions github-actions Bot added triage-framework Should be looked at in framework triage p: material_ui labels Jul 16, 2026

@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 adds a README file to the packages/material_ui/tool/gen_defaults/temporarily_excluded/ directory, providing instructions for migrating templates and generated code. The review identified two improvements: clarifying the naming convention placeholders used in the instructions and correcting the Git command sequence for staging and committing moved files.

Comment thread packages/material_ui/tool/gen_defaults/temporarily_excluded/README.md Outdated
elliette and others added 4 commits July 16, 2026 12:57
…ADME.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…ADME.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

@QuncCccccc QuncCccccc 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 with minor comments.


// const ActionChipTemplateM3().generateFile(verbose: verbose);
const AppBarTemplateM3().generateFile(verbose: verbose);
// const BadgeTemplateM3().generateFile(verbose: verbose);

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.

Smart way to avoid conflict!!

@elliette elliette changed the title [material_ui] Add a README for migrating gen_defaults templates [material_ui] Add a README and placeholders for migrating gen_defaults templates Jul 16, 2026
@elliette
elliette merged commit 04d1adb into flutter:m3e_migration Jul 16, 2026
3 of 5 checks passed
auto-submit Bot pushed a commit that referenced this pull request Sep 2, 2026
…s templates (#12714)

Work towards flutter/flutter#191088

**Ports over #12220 which landed on the `m3e_migration` feature branch.**

### Original PR description

Work towards flutter/flutter#187899

* Adds a README on how to migrate the gen_defaults templates, to be used by contributors or agents.
* Adds placeholders (referenced in README) for templates to prevent merge conflicts while migrating.

## Pre-Review Checklist

**Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot 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.

[^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
victorsanni pushed a commit to victorsanni/packages that referenced this pull request Sep 9, 2026
…s templates (flutter#12714)

Work towards flutter/flutter#191088

**Ports over flutter#12220 which landed on the `m3e_migration` feature branch.**

### Original PR description

Work towards flutter/flutter#187899

* Adds a README on how to migrate the gen_defaults templates, to be used by contributors or agents.
* Adds placeholders (referenced in README) for templates to prevent merge conflicts while migrating.

## Pre-Review Checklist

**Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot 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.

[^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
victorsanni pushed a commit to victorsanni/packages that referenced this pull request Sep 9, 2026
…s templates (flutter#12714)

Work towards flutter/flutter#191088

**Ports over flutter#12220 which landed on the `m3e_migration` feature branch.**

### Original PR description

Work towards flutter/flutter#187899

* Adds a README on how to migrate the gen_defaults templates, to be used by contributors or agents.
* Adds placeholders (referenced in README) for templates to prevent merge conflicts while migrating.

## Pre-Review Checklist

**Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot 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.

[^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD p: material_ui triage-framework Should be looked at in framework triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants