Skip to content

Commit f584b03

Browse files
authored
Add and apply lints for imports (#1940)
1 parent c453ebe commit f584b03

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

52 files changed

+214
-247
lines changed

dwds/analysis_options.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,7 @@ analyzer:
1515

1616
linter:
1717
rules:
18+
- always_use_package_imports
19+
- directives_ordering
1820
- prefer_final_locals
1921
- unawaited_futures

dwds/debug_extension_mv3/web/background.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,10 @@ import 'dart:html';
1111
import 'package:dwds/data/debug_info.dart';
1212
import 'package:js/js.dart';
1313

14-
import 'data_types.dart';
15-
import 'debug_session.dart';
1614
import 'chrome_api.dart';
1715
import 'cross_extension_communication.dart';
16+
import 'data_types.dart';
17+
import 'debug_session.dart';
1818
import 'lifeline_ports.dart';
1919
import 'logger.dart';
2020
import 'messaging.dart';

dwds/debug_extension_mv3/web/devtools.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,9 @@
66
library devtools;
77

88
import 'dart:html';
9-
import 'package:js/js.dart';
9+
1010
import 'package:dwds/data/debug_info.dart';
11+
import 'package:js/js.dart';
1112

1213
import 'chrome_api.dart';
1314
import 'logger.dart';

dwds/debug_extension_mv3/web/messaging.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ import 'dart:convert';
99

1010
import 'package:js/js.dart';
1111

12-
import 'data_serializers.dart';
1312
import 'chrome_api.dart';
13+
import 'data_serializers.dart';
1414
import 'logger.dart';
1515

1616
enum Script {

dwds/debug_extension_mv3/web/web_api.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.
44
import 'dart:html';
5+
import 'dart:js_util' as js_util;
56

67
import 'package:js/js.dart';
7-
import 'dart:js_util' as js_util;
88

99
@JS()
1010
// ignore: non_constant_identifier_names

dwds/lib/dart_web_debug_service.dart

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -4,24 +4,23 @@
44

55
import 'dart:async';
66

7+
import 'package:dwds/data/build_result.dart';
8+
import 'package:dwds/src/connections/app_connection.dart';
9+
import 'package:dwds/src/connections/debug_connection.dart';
10+
import 'package:dwds/src/events.dart';
11+
import 'package:dwds/src/handlers/dev_handler.dart';
12+
import 'package:dwds/src/handlers/injector.dart';
13+
import 'package:dwds/src/handlers/socket_connections.dart';
14+
import 'package:dwds/src/loaders/strategy.dart';
15+
import 'package:dwds/src/readers/asset_reader.dart';
16+
import 'package:dwds/src/servers/devtools.dart';
17+
import 'package:dwds/src/servers/extension_backend.dart';
18+
import 'package:dwds/src/services/expression_compiler.dart';
719
import 'package:logging/logging.dart';
820
import 'package:shelf/shelf.dart';
921
import 'package:sse/server/sse_handler.dart';
1022
import 'package:webkit_inspection_protocol/webkit_inspection_protocol.dart';
1123

12-
import 'data/build_result.dart';
13-
import 'src/connections/app_connection.dart';
14-
import 'src/connections/debug_connection.dart';
15-
import 'src/events.dart';
16-
import 'src/handlers/dev_handler.dart';
17-
import 'src/handlers/injector.dart';
18-
import 'src/handlers/socket_connections.dart';
19-
import 'src/loaders/strategy.dart';
20-
import 'src/readers/asset_reader.dart';
21-
import 'src/servers/devtools.dart';
22-
import 'src/servers/extension_backend.dart';
23-
import 'src/services/expression_compiler.dart';
24-
2524
typedef ConnectionProvider = Future<ChromeConnection> Function();
2625

2726
/// The Dart Web Debug Service.

dwds/lib/src/connections/app_connection.dart

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@
55
import 'dart:async';
66
import 'dart:convert';
77

8-
import '../../data/connect_request.dart';
9-
import '../../data/run_request.dart';
10-
import '../../data/serializers.dart';
11-
import '../handlers/socket_connections.dart';
8+
import 'package:dwds/data/connect_request.dart';
9+
import 'package:dwds/data/run_request.dart';
10+
import 'package:dwds/data/serializers.dart';
11+
import 'package:dwds/src/handlers/socket_connections.dart';
1212

1313
/// A connection between the application loaded in the browser and DWDS.
1414
class AppConnection {

dwds/lib/src/connections/debug_connection.dart

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,10 @@
44

55
import 'dart:async';
66

7+
import 'package:dwds/src/services/app_debug_services.dart';
8+
import 'package:dwds/src/services/chrome_proxy_service.dart';
79
import 'package:vm_service/vm_service.dart';
810

9-
import '../services/app_debug_services.dart';
10-
import '../services/chrome_proxy_service.dart';
11-
1211
/// A debug connection between the application in the browser and DWDS.
1312
///
1413
/// Supports debugging your running application through the Dart VM Service

dwds/lib/src/debugging/classes.dart

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,14 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.import 'dart:async';
44

5+
import 'package:dwds/src/debugging/metadata/class.dart';
6+
import 'package:dwds/src/loaders/strategy.dart';
7+
import 'package:dwds/src/services/chrome_debug_exception.dart';
8+
import 'package:dwds/src/utilities/domain.dart';
9+
import 'package:dwds/src/utilities/shared.dart';
510
import 'package:vm_service/vm_service.dart';
611
import 'package:webkit_inspection_protocol/webkit_inspection_protocol.dart';
712

8-
import '../../src/services/chrome_debug_exception.dart';
9-
import '../loaders/strategy.dart';
10-
import '../utilities/domain.dart';
11-
import '../utilities/shared.dart';
12-
import 'metadata/class.dart';
13-
1413
/// Keeps track of Dart classes available in the running application.
1514
class ClassHelper extends Domain {
1615
/// Map of class ID to [Class].

dwds/lib/src/debugging/dart_scope.dart

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,10 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.
44

5+
import 'package:dwds/src/debugging/debugger.dart';
6+
import 'package:dwds/src/utilities/objects.dart';
57
import 'package:webkit_inspection_protocol/webkit_inspection_protocol.dart';
68

7-
import '../utilities/objects.dart';
8-
import 'debugger.dart';
9-
109
// TODO(sdk/issues/44262) - use an alternative way to identify synthetic
1110
// variables.
1211
final ddcTemporaryVariableRegExp = RegExp(r'^(t[0-9]+\$?[0-9]*|__t[\$\w*]+)$');

0 commit comments

Comments
 (0)