Skip to content

[flutter_tools] Replace usages of package:dds/dap.dart with package:dap_adapters/dap_adapters.dart - #190667

Open
bkonyi wants to merge 5 commits into
flutter:masterfrom
bkonyi:migrate_dap_adapter_usage
Open

[flutter_tools] Replace usages of package:dds/dap.dart with package:dap_adapters/dap_adapters.dart#190667
bkonyi wants to merge 5 commits into
flutter:masterfrom
bkonyi:migrate_dap_adapter_usage

Conversation

@bkonyi

@bkonyi bkonyi commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Migrate Debug Adapter Protocol (DAP) adapter imports in flutter_tools from package:dds/dap.dart to package:dap_adapters/dap_adapters.dart.

…ap_adapters/dap_adapters.dart

Migrate Debug Adapter Protocol (DAP) adapter imports in flutter_tools from package:dds/dap.dart to package:dap_adapters/dap_adapters.dart. Also update package:flutter_tools pubspec.yaml dependencies and pubspec checksum.
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Aug 6, 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 several package dependencies across the repository, including upgrades to meta, vector_math, and video_player. It also introduces the dap_adapters package and migrates debug adapter imports from package:dds/dap.dart to package:dap_adapters/dap_adapters.dart. Feedback was provided to maintain alphabetical ordering of dependencies in packages/flutter_tools/pubspec.yaml by placing dap_adapters before dds.

Comment thread packages/flutter_tools/pubspec.yaml Outdated
Comment on lines +16 to +17
dds: 5.3.0
dap_adapters: 1.0.0

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.

medium

To maintain alphabetical order of dependencies in pubspec.yaml, dap_adapters should be placed before dds.

  dap_adapters: 1.0.0
  dds: 5.3.0

bkonyi added 3 commits August 6, 2026 16:39
…sage

# Conflicts:
#	dev/integration_tests/hook_user_defines/pubspec.yaml
#	packages/flutter_tools/pubspec.yaml
#	pubspec.yaml
@bkonyi
bkonyi changed the base branch from main to master August 6, 2026 18:35
@github-actions github-actions Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Aug 6, 2026
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.

1 participant