Skip to content

Replace pub run mentions with dart run#181317

Merged
auto-submit[bot] merged 2 commits intoflutter:masterfrom
sigurdm:remove-pub-run
Jan 27, 2026
Merged

Replace pub run mentions with dart run#181317
auto-submit[bot] merged 2 commits intoflutter:masterfrom
sigurdm:remove-pub-run

Conversation

@sigurdm
Copy link
Contributor

@sigurdm sigurdm commented Jan 22, 2026

Following up on dart-lang/pub#4737, this PR replaces deprecated pub run commands with dart run.

@github-actions github-actions bot added tool Affects the "flutter" command-line tool. See also t: labels. package flutter/packages repository. See also p: labels. team-ecosystem Owned by Ecosystem team d: docs/ flutter/flutter/docs, for contributors labels Jan 22, 2026
@sigurdm sigurdm marked this pull request as ready for review January 22, 2026 13:20
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 updates deprecated pub run commands to dart run across documentation and tool messages. The changes are correct and align with the move away from pub run. I have one suggestion for a potential follow-up to ensure consistency between documentation and implementation in one of the commands.

'differences are that the "dart:ui" library is not available and that tests '
'run faster. This is helpful for testing libraries that do not depend on any '
'packages from the Flutter SDK. It is equivalent to "pub run test".';
'packages from the Flutter SDK. It is equivalent to "dart run test".';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

While this description is correctly updated to mention dart run test, the implementation of runCommand() for PackagesTestCommand (on line 129) still uses pub.batch(['run', 'test', ...]), which effectively calls the deprecated dart pub run test. To make the implementation consistent with the updated description, it should be changed to execute dart run test directly. This might be best addressed in a follow-up PR as it's outside the current diff.

Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating these!

@parlough parlough changed the title Replace pub run with dart run Replace pub run mentions with dart run Jan 27, 2026
@parlough parlough added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 27, 2026
@auto-submit auto-submit bot added this pull request to the merge queue Jan 27, 2026
Merged via the queue into flutter:master with commit c909c58 Jan 27, 2026
141 checks passed
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jan 27, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 27, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 27, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 27, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 27, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 27, 2026
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jan 27, 2026
flutter/flutter@7165649...dfd92b7

2026-01-27 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Fixes metal vec3 uniform padding (#181340)" (flutter/flutter#181552)
2026-01-27 [email protected] Change update_engine_version documenation (flutter/flutter#181508)
2026-01-27 [email protected] Roll Packages from e712bfa to e37af11 (7 revisions) (flutter/flutter#181544)
2026-01-27 [email protected] Roll FreeType to 2.14.1 (flutter/flutter#181428)
2026-01-27 [email protected] Roll Skia from 566d202962d3 to c2754838b077 (3 revisions) (flutter/flutter#181536)
2026-01-27 [email protected] [Material] modernize time picker components with Dart 3 switch expressions (flutter/flutter#181356)
2026-01-27 [email protected] Roll Skia from 55a87b50a368 to 566d202962d3 (1 revision) (flutter/flutter#181528)
2026-01-27 [email protected] Roll Dart SDK from f55d4538469a to 4c7cb0a1d07d (2 revisions) (flutter/flutter#181526)
2026-01-27 [email protected] Replace `pub run` mentions with `dart run` (flutter/flutter#181317)
2026-01-27 [email protected] Roll Skia from 1a15ed6b17f2 to 55a87b50a368 (1 revision) (flutter/flutter#181523)
2026-01-27 [email protected] Marks linux_chrome_dev_mode to be unflaky (flutter/flutter#181352)
2026-01-27 [email protected] Roll Skia from f2c1aa140b79 to 1a15ed6b17f2 (2 revisions) (flutter/flutter#181514)
2026-01-27 [email protected] Fix Range Slider issue where indescrete ranges lead to out of range r… (flutter/flutter#181082)
2026-01-26 [email protected] Roll Dart SDK from ba23b5ed0a97 to f55d4538469a (2 revisions) (flutter/flutter#181512)
2026-01-26 [email protected] Add `alignment` to `SizeTransition` (flutter/flutter#177895)
2026-01-26 [email protected] Roll Fuchsia Linux SDK from T4qTEa3T5CCSCIoJY... to akraNGn2lw4T1msgZ... (flutter/flutter#181509)
2026-01-26 [email protected] Roll Skia from be280d242a60 to f2c1aa140b79 (3 revisions) (flutter/flutter#181504)
2026-01-26 [email protected] Fixes metal vec3 uniform padding (flutter/flutter#181340)
2026-01-26 [email protected] Fixes duplicated import in accessibility test library (flutter/flutter#181506)
2026-01-26 [email protected] [Reland] Don't strip symbols from `libapp.so` on android by default (flutter/flutter#181275)
2026-01-26 [email protected] Fix Gradle path in Android Platform Embedder README. (flutter/flutter#181501)
2026-01-26 [email protected] Remove unused `ActivityLifecycleListener` class (flutter/flutter#181406)

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 [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
flutter-zl pushed a commit to flutter-zl/flutter that referenced this pull request Feb 10, 2026
Following up on dart-lang/pub#4737, this PR
replaces deprecated `pub run` commands with `dart run`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

d: docs/ flutter/flutter/docs, for contributors package flutter/packages repository. See also p: labels. team-ecosystem Owned by Ecosystem team 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