Skip to content

Require Dart SDK 3.7 and drop Dart 2 tooling - #474

Merged
btr-rmconsole-5[bot] merged 3 commits into
masterfrom
dart3
Sep 17, 2026
Merged

btr-rmconsole-5[bot] merged 3 commits into
masterfrom
dart3

Conversation

@robbecker-wf

Copy link
Copy Markdown
Member

Problem / Feature

dart_dev still advertised Dart 2.19 support and branched on SDK commands (dartanalyzer, dartfmt) and webdev 2.x that no longer exist. Dart 3.7+ also requires an explicit dart format --language-version, which the current minimum SDK cannot provide.

Solution

Raise the SDK constraint to >=3.7.0 <4.0.0 and always run dart analyze / dart format. Formatter.dartfmt now maps to dart format (and is the default); AnalyzeTool.useDartAnalyze is ignored. Serve accepts globally activated webdev >=3.0.0 <5.0.0. TuneupCheckTool is removed. CI tests 3.7.0 and stable; the format check is commented out until a dedicated format PR.

Areas of Impact

  • SDK / pubspec constraint and CI matrix
  • AnalyzeTool, FormatTool, WebdevServeTool
  • Public API: TuneupCheckTool removed; dartfmt / useDartAnalyze deprecated
  • Docs, tests, and fixtures updated to Dart 3.7

Test Overview (QA Instructions)

  • dart analyze is clean
  • dart test passes on Dart 3.7+
  • CI jobs run on 3.7.0 and stable (ubuntu + windows)
  • Confirm the format --check step is still commented in .github/workflows/ci.yml
  • Consumers on Dart <3.7 cannot resolve this version (expected breaking change)
  • ddev serve accepts webdev 3.x or 4.x; TuneupCheckTool is no longer exported

Made with Cursor

Raise the SDK floor so dart format --language-version works, always use
dart analyze/format, allow webdev 3.x and 4.x, and remove TuneupCheckTool.

Co-authored-by: Cursor <cursoragent@cursor.com>
Widen the constraint so Dart 3.13+ can resolve analyzer 14.x, and fail CI
if stable does not actually get that version.

Co-authored-by: Cursor <cursoragent@cursor.com>
@robbecker-wf
robbecker-wf added this pull request to stack #476 September 17, 2026 19:38
@robbecker-wf
robbecker-wf marked this pull request as ready for review September 17, 2026 21:16
@robbecker-wf
robbecker-wf requested a review from a team as a code owner September 17, 2026 21:16
@robbecker-wf

Copy link
Copy Markdown
Member Author

QA +1 CI passes @Workiva/release-management-p

@rmconsole-wf rmconsole-wf 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.

+1 from RM

@btr-rmconsole-5
btr-rmconsole-5 Bot merged commit 5472679 into master Sep 17, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants