Skip to content

[rfw] Make widget builders work with loops - #8650

Merged
auto-submit[bot] merged 1 commit into
flutter:mainfrom
tugorez:builder-loops
Mar 5, 2025
Merged

auto-submit[bot] merged 1 commit into
flutter:mainfrom
tugorez:builder-loops

Conversation

@tugorez

@tugorez tugorez commented Feb 17, 2025

Copy link
Copy Markdown
Contributor

Make widget builders work with loops

Fixes flutter/flutter#161544

Pre-launch Checklist

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

@tugorez
tugorez requested a review from Hixie as a code owner February 17, 2025 20:43
@github-actions github-actions Bot added the p: rfw Remote Flutter Widgets label Feb 17, 2025
@tugorez tugorez self-assigned this Feb 17, 2025
@tugorez
tugorez force-pushed the builder-loops branch 2 times, most recently from 2387401 to c881159 Compare February 17, 2025 23:08
@stuartmorgan-g stuartmorgan-g changed the title Make widget builders work with loops [rfw] Make widget builders work with loops Feb 18, 2025
@tugorez

tugorez commented Feb 26, 2025

Copy link
Copy Markdown
Contributor Author

@Hixie PTAL when you get a chance. It seems I forgot to consider loops when landing the builders implementation

@ditman

ditman commented Mar 5, 2025

Copy link
Copy Markdown
Member

I'm interested in how this review pans out.

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

Seems fine to me, but I must admit I'm no longer sufficiently familiar with this code to say this with confidence.

@tugorez tugorez added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 5, 2025
@auto-submit
auto-submit Bot merged commit 7b635eb into flutter:main Mar 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 6, 2025
github-merge-queue Bot pushed a commit to flutter/flutter that referenced this pull request Mar 6, 2025
flutter/packages@abba683...fc9d5ca

2025-03-06 [email protected] manual roll with video player test
disabled (flutter/packages#8803)
2025-03-05 [email protected] [rfw] Make widget builders
work with loops (flutter/packages#8650)
2025-03-05 [email protected] [shared_preferences_android] dependabot
takeover datastore (flutter/packages#8784)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected] on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
androidseb pushed a commit to androidseb/packages that referenced this pull request Jun 8, 2025
FMorschel pushed a commit to FMorschel/packages that referenced this pull request Jun 9, 2025
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App p: rfw Remote Flutter Widgets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[RFW] Loops are not working in widget builder scopes

3 participants