Skip to content

Conversation

@bkonyi
Copy link
Contributor

@bkonyi bkonyi commented Nov 17, 2025

This reverts commit e983e4b.

@bkonyi bkonyi requested a review from jtmcdole November 17, 2025 20:48
@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Nov 17, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

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 reverts a previous change that prevented flutter clean from deleting the .dart_tool/widget_preview_scaffold directory. The changes remove the special handling logic from the clean command, restoring the behavior of deleting the entire .dart_tool directory. The associated tests have also been reverted to reflect this behavior.

@jtmcdole jtmcdole added this pull request to the merge queue Nov 18, 2025
Merged via the queue into master with commit 731b367 Nov 18, 2025
144 checks passed
@jtmcdole jtmcdole deleted the revert_e983e4bd81f29b17215057fa5c9f46f96cbaf183 branch November 18, 2025 11:01
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 18, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 18, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 18, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 18, 2025
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Nov 18, 2025
flutter/flutter@cc14ef5...cb7b7df

2025-11-18 15619084+vashworth@users.noreply.github.com Enable UIScene Migration and update create templates (flutter/flutter#178700)
2025-11-18 Minsuk Jung Fix #160622: change containsWatchConpanion function to detect companion watch apps defined by only the project info file. (flutter/flutter#176832)
2025-11-18 engine-flutter-autoroll@skia.org Roll Skia from 614e71550fc3 to ca906091e199 (2 revisions) (flutter/flutter#178716)
2025-11-18 bkonyi@google.com Revert "[ Tool ] Don't delete `.dart_tool/widget_preview_scaffold` during `flutter clean` (#175664)" (flutter/flutter#178672)
2025-11-18 6655696+guidezpl@users.noreply.github.com Add missing flutter_lints dev dependencies (flutter/flutter#178105)
2025-11-18 engine-flutter-autoroll@skia.org Roll Skia from ec2f626cdcad to 614e71550fc3 (3 revisions) (flutter/flutter#178708)
2025-11-18 engine-flutter-autoroll@skia.org Roll Dart SDK from a8ad764281e3 to 312845b06afc (1 revision) (flutter/flutter#178704)
2025-11-18 engine-flutter-autoroll@skia.org Roll Skia from d7268f8245f2 to ec2f626cdcad (1 revision) (flutter/flutter#178703)
2025-11-18 116356835+AbdeMohlbi@users.noreply.github.com Refactor SnackBar behavior selection example to use `RadioGroup` (flutter/flutter#178618)
2025-11-18 zhongliu88889@gmail.com Add framework-side hitTestBehavior support for Semantics widget and apply to ModalRoute (flutter/flutter#177570)
2025-11-18 huy@nevercode.io Fix deprecation warning in some API examples using RadioListTile (flutter/flutter#178635)
2025-11-18 engine-flutter-autoroll@skia.org Roll Skia from 47fd0d9b1044 to d7268f8245f2 (6 revisions) (flutter/flutter#178695)
2025-11-18 engine-flutter-autoroll@skia.org Roll Dart SDK from cf94632d94a1 to a8ad764281e3 (1 revision) (flutter/flutter#178694)
2025-11-18 69134234+sutes-work@users.noreply.github.com [fuchsia] Add wrapper for zx_iob_writev (flutter/flutter#178626)
2025-11-17 31859944+LongCatIsLooong@users.noreply.github.com Make a11y `computeChildGeometry` slightly faster (flutter/flutter#177477)
2025-11-17 bruno.leroux@gmail.com Fix DropdownMenu width when decorationBuilder provides label (flutter/flutter#178465)
2025-11-17 bruno.leroux@gmail.com Add DropdownMenuFormField.decorationBuilder (flutter/flutter#178640)
2025-11-17 engine-flutter-autoroll@skia.org Roll Skia from 84c83c0dfb4a to 47fd0d9b1044 (4 revisions) (flutter/flutter#178673)
2025-11-17 116356835+AbdeMohlbi@users.noreply.github.com Small cleanup in `AndroidTouchProcessor.java‎` (flutter/flutter#178574)
2025-11-17 116356835+AbdeMohlbi@users.noreply.github.com Remove unnecessary `final` modifier in `StandardMessageCodec.java‎` (flutter/flutter#178598)

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
Please CC louisehsu@google.com,stuartmorgan@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: 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
IvoneDjaja pushed a commit to IvoneDjaja/flutter that referenced this pull request Nov 22, 2025
mboetger pushed a commit to mboetger/flutter that referenced this pull request Dec 2, 2025
reidbaker pushed a commit to AbdeMohlbi/flutter that referenced this pull request Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants