Skip to content

Commit 4bc2462

Browse files
committed
Run dcm check-unused-code on the devtools_extensions package
1 parent d6079ca commit 4bc2462

5 files changed

Lines changed: 1 addition & 19 deletions

File tree

.github/workflows/build.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ jobs:
8686
- name: Run dcm checks on packages
8787
# TODO(https://github.com/flutter/devtools/issues/9906): run on all DevTools packages.
8888
run: |
89-
dcm check-unused-code packages/devtools_app --exclude-public-api
89+
dcm check-unused-code packages/devtools_app packages/devtools_extensions --exclude-public-api
9090
9191
test-packages:
9292
name: ${{ matrix.os }} ${{ matrix.package }} test

packages/devtools_extensions/integration_test/test/simulated_environment_test.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import 'package:integration_test/integration_test.dart';
1515
// dart run integration_test/run_tests.dart --target=integration_test/test/simulated_environment_test.dart
1616

1717
const safePumpDuration = Duration(seconds: 3);
18-
const longPumpDuration = Duration(seconds: 6);
1918

2019
void main() {
2120
IntegrationTestWidgetsFlutterBinding.ensureInitialized();

packages/devtools_extensions/lib/src/template/_simulated_devtools_environment/connection_ui/_connect.dart

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ part of '../_simulated_devtools_environment.dart';
66

77
class _UriConnectionDisplay extends StatelessWidget {
88
const _UriConnectionDisplay({
9-
required this.simController,
109
required this.connected,
1110
required this.connectedLabel,
1211
required this.disconnectedLabel,
@@ -20,7 +19,6 @@ class _UriConnectionDisplay extends StatelessWidget {
2019
@visibleForTesting
2120
static const totalControlsHeight = 45.0;
2221

23-
final SimulatedDevToolsController simController;
2422
final bool connected;
2523
final String connectedLabel;
2624
final String disconnectedLabel;
@@ -36,14 +34,12 @@ class _UriConnectionDisplay extends StatelessWidget {
3634
height: totalControlsHeight,
3735
child: connected
3836
? _ConnectedDisplay(
39-
simController: simController,
4037
label: connectedLabel,
4138
onDisconnect: onDisconnect,
4239
currentConnection: currentConnection,
4340
help: help,
4441
)
4542
: _DisconnectedDisplay(
46-
simController: simController,
4743
label: disconnectedLabel,
4844
hint: disconnectedHint,
4945
onConnect: onConnect,
@@ -55,14 +51,12 @@ class _UriConnectionDisplay extends StatelessWidget {
5551

5652
class _ConnectedDisplay extends StatelessWidget {
5753
const _ConnectedDisplay({
58-
required this.simController,
5954
required this.label,
6055
required this.currentConnection,
6156
required this.onDisconnect,
6257
this.help,
6358
});
6459

65-
final SimulatedDevToolsController simController;
6660
final String label;
6761
final String Function() currentConnection;
6862
final VoidCallback onDisconnect;
@@ -98,14 +92,12 @@ class _ConnectedDisplay extends StatelessWidget {
9892

9993
class _DisconnectedDisplay extends StatefulWidget {
10094
const _DisconnectedDisplay({
101-
required this.simController,
10295
required this.label,
10396
required this.hint,
10497
required this.onConnect,
10598
this.help,
10699
});
107100

108-
final SimulatedDevToolsController simController;
109101
final String label;
110102
final String hint;
111103
final void Function(String) onConnect;

packages/devtools_extensions/lib/src/template/_simulated_devtools_environment/connection_ui/_dtd_connect.dart

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,13 @@ class DTDConnectionDisplay extends StatelessWidget {
1313
required this.onConnectionChange,
1414
});
1515

16-
@visibleForTesting
17-
static const totalControlsHeight = 45.0;
18-
@visibleForTesting
19-
static const totalControlsWidth = 415.0;
20-
2116
final SimulatedDevToolsController simController;
2217
final bool connected;
2318
final Future<void> Function(String?) onConnectionChange;
2419

2520
@override
2621
Widget build(BuildContext context) {
2722
return _UriConnectionDisplay(
28-
simController: simController,
2923
connected: connected,
3024
connectedLabel: 'Dart Tooling Daemon connection: ',
3125
disconnectedLabel: 'Dart Tooling Daemon Connection',

packages/devtools_extensions/lib/src/template/_simulated_devtools_environment/connection_ui/_vm_service_connect.dart

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@ class VmServiceConnectionDisplay extends StatelessWidget {
1212
required this.connected,
1313
});
1414

15-
@visibleForTesting
16-
static const totalControlsHeight = 45.0;
1715
@visibleForTesting
1816
static const totalControlsWidth = 415.0;
1917

@@ -23,7 +21,6 @@ class VmServiceConnectionDisplay extends StatelessWidget {
2321
@override
2422
Widget build(BuildContext context) {
2523
return _UriConnectionDisplay(
26-
simController: simController,
2724
connected: connected,
2825
connectedLabel: 'Debugging: ',
2926
disconnectedLabel: 'Dart VM Service Connection',

0 commit comments

Comments
 (0)