Skip to content

Warn users on potential removal of dart:html in dry run - #192710

Merged
auto-submit[bot] merged 2 commits into
flutter:masterfrom
kevmoo:warn-dart-html-removal
Sep 14, 2026
Merged

auto-submit[bot] merged 2 commits into
flutter:masterfrom
kevmoo:warn-dart-html-removal

Conversation

@kevmoo

@kevmoo kevmoo commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Update the dry-run warning messages in flutter_tools to clearly
communicate that dart:html, dart:js, and legacy JS interop libraries
are deprecated and planned for removal from the Dart SDK in a future
release, encouraging users to migrate to package:web and dart:js_interop.

Fixes #190562

Update the dry-run warning messages in flutter_tools to clearly
communicate that dart:html, dart:js, and legacy JS interop libraries
are deprecated and planned for removal from the Dart SDK in a future
release, encouraging users to migrate to package:web and dart:js_interop.
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Sep 13, 2026
@github-actions github-actions Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Sep 13, 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 updates the deprecation warning messages for legacy web imports (such as dart:html and dart:js) during WebAssembly compilation and dry runs, and updates the corresponding tests. The reviewer suggests extracting the duplicated warning message into a single constant to avoid duplicating state, adhering to the Flutter style guide.

Comment thread packages/flutter_tools/lib/src/build_system/targets/web.dart Outdated
Comment thread packages/flutter_tools/lib/src/build_system/targets/web.dart Outdated
Addresses review feedback on flutter#192710 by extracting the duplicated
warning message into .
@kevmoo
kevmoo requested a review from srujzs September 13, 2026 03:55

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

Thanks Kevin!

@kevmoo kevmoo added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 14, 2026
@auto-submit
auto-submit Bot added this pull request to the merge queue Sep 14, 2026
Merged via the queue into flutter:master with commit 79c1e5e Sep 14, 2026
23 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Sep 14, 2026
@kevmoo
kevmoo deleted the warn-dart-html-removal branch September 15, 2026 02:58
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Sep 16, 2026
flutter/flutter@8b3e8f5...27fec0e

2026-09-14 [email protected] Add dedicated rubber band spring for overscroll recovery in BouncingScrollPhysics (flutter/flutter#187568)
2026-09-14 [email protected] Roll Skia from 83c984a6daf5 to 59904977fcbb (8 revisions) (flutter/flutter#192767)
2026-09-14 [email protected] Request unbuffered dispatch for platform view motion events after `_PlatformViewGestureRecognizer` rejects gesture (flutter/flutter#192259)
2026-09-14 [email protected] Roll Fuchsia GN SDK from ZsxUUK2SfA-TrNvb8... to PDJz3b3CL31uoLOkD... (flutter/flutter#192440)
2026-09-14 [email protected] ci(bringup): cleanup some bringup (flutter/flutter#192752)
2026-09-14 [email protected] Warn users on potential removal of dart:html in dry run (flutter/flutter#192710)
2026-09-14 [email protected] docs: migrate design document guidance to flutter/rfc (flutter/flutter#192648)
2026-09-14 [email protected] Sync CHANGELOG.md from stable (flutter/flutter#192499)
2026-09-14 [email protected] Roll Skia from 7a6fd7ff72d5 to 83c984a6daf5 (1 revision) (flutter/flutter#192745)
2026-09-14 [email protected] RELAND: [devicelab] Clean up Chrome profile directories and trap abort signals in web benchmarks (flutter/flutter#192655)
2026-09-14 [email protected] fix(web): hide --web-content-hash until content hashing is complete (flutter/flutter#192663)
2026-09-14 [email protected] Roll Skia from 527751088719 to 7a6fd7ff72d5 (2 revisions) (flutter/flutter#192725)
2026-09-14 [email protected] Roll Skia from eaf9f82813eb to 527751088719 (1 revision) (flutter/flutter#192722)
2026-09-14 [email protected] Roll Fuchsia Linux SDK from AQu85JoTHCMET3Q8G... to 7R1l5MhSZZUr1Q6D1... (flutter/flutter#192719)
2026-09-14 [email protected] [Linux] Clear the frame when there is nothing to render (flutter/flutter#192461)

If this roll has caused a breakage, revert this CL and set the roller
to dry run mode using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected] 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Warn users on potential removal of dart:html (and similar libraries) in dry run

2 participants