diff --git a/packages/flutter_tools/bin/tool_backend.dart b/packages/flutter_tools/bin/tool_backend.dart index e63228f08f8..eb4b265129e 100644 --- a/packages/flutter_tools/bin/tool_backend.dart +++ b/packages/flutter_tools/bin/tool_backend.dart @@ -2,8 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -import 'dart:io'; // ignore: dart_io_import. -import 'package:path/path.dart' as path; // ignore: package_path_import. +import 'dart:io'; // ignore_for_flutter: dart_io_import. +import 'package:path/path.dart' as path; // ignore_for_flutter: package_path_import. /// Executes the required flutter tasks for a desktop build. Future main(List arguments) async { diff --git a/packages/flutter_tools/lib/src/build_runner/build_script.dart b/packages/flutter_tools/lib/src/build_runner/build_script.dart index 757206c14df..c52076e9d39 100644 --- a/packages/flutter_tools/lib/src/build_runner/build_script.dart +++ b/packages/flutter_tools/lib/src/build_runner/build_script.dart @@ -3,6 +3,10 @@ // found in the LICENSE file. // ignore_for_file: implementation_imports +// The "// ignore:" comments found in strings in this file are counted as _actual_ ignore comments. +// Since the "// ignore:" comments found in strings don't actually ignore anythin in _this_ file, +// we must allow them to be "unused." See https://github.com/dart-lang/sdk/issues/39607. +// ignore_for_file: unused_ignore_code import 'dart:async'; import 'dart:isolate'; @@ -22,7 +26,7 @@ import 'package:build_test/src/debug_test_builder.dart'; import 'package:build_web_compilers/build_web_compilers.dart'; import 'package:build_web_compilers/builders.dart'; import 'package:build_web_compilers/src/dev_compiler_bootstrap.dart'; -import 'package:path/path.dart' as path; // ignore: package_path_import +import 'package:path/path.dart' as path; // ignore_for_flutter: package_path_import import 'package:test_core/backend.dart'; // ignore: deprecated_member_use const String ddcBootstrapExtension = '.dart.bootstrap.js'; diff --git a/packages/flutter_tools/lib/src/build_runner/web_compilation_delegate.dart b/packages/flutter_tools/lib/src/build_runner/web_compilation_delegate.dart index 4f8ae4f9607..380cae00aae 100644 --- a/packages/flutter_tools/lib/src/build_runner/web_compilation_delegate.dart +++ b/packages/flutter_tools/lib/src/build_runner/web_compilation_delegate.dart @@ -2,9 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -// ignore_for_file: implementation_imports import 'dart:async'; -import 'dart:io' as io; // ignore: dart_io_import +import 'dart:io' as io; // ignore_for_flutter: dart_io_import import 'package:build/build.dart'; import 'package:build_daemon/client.dart'; diff --git a/packages/flutter_tools/lib/src/test/flutter_platform.dart b/packages/flutter_tools/lib/src/test/flutter_platform.dart index 3952fcc08ec..428440bf8e3 100644 --- a/packages/flutter_tools/lib/src/test/flutter_platform.dart +++ b/packages/flutter_tools/lib/src/test/flutter_platform.dart @@ -159,8 +159,8 @@ String generateTestBootstrap({ final StringBuffer buffer = StringBuffer(); buffer.write(''' import 'dart:async'; -import 'dart:convert'; // ignore: dart_convert_import -import 'dart:io'; // ignore: dart_io_import +import 'dart:convert'; // ignore_for_flutter: dart_convert_import +import 'dart:io'; // ignore_for_flutter: dart_io_import import 'dart:isolate'; import 'package:flutter_test/flutter_test.dart'; diff --git a/packages/flutter_tools/lib/src/test/flutter_web_platform.dart b/packages/flutter_tools/lib/src/test/flutter_web_platform.dart index 210164e14e1..aa939fbb278 100644 --- a/packages/flutter_tools/lib/src/test/flutter_web_platform.dart +++ b/packages/flutter_tools/lib/src/test/flutter_web_platform.dart @@ -9,7 +9,7 @@ import 'dart:typed_data'; import 'package:async/async.dart'; import 'package:http_multi_server/http_multi_server.dart'; -import 'package:path/path.dart' as p; // ignore: package_path_import +import 'package:path/path.dart' as p; // ignore_for_flutter: package_path_import import 'package:pool/pool.dart'; import 'package:shelf/shelf.dart' as shelf; import 'package:shelf/shelf_io.dart' as shelf_io; diff --git a/packages/flutter_tools/test/general.shard/dart/sdk_validation_test.dart b/packages/flutter_tools/test/general.shard/dart/sdk_validation_test.dart index 90f35353771..eb4ca4087b5 100644 --- a/packages/flutter_tools/test/general.shard/dart/sdk_validation_test.dart +++ b/packages/flutter_tools/test/general.shard/dart/sdk_validation_test.dart @@ -2,6 +2,10 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. +// The "// ignore:" comments found in strings in this file are counted as _actual_ ignore comments. +// Since the "// ignore:" comments found in strings don't actually ignore anythin in _this_ file, +// we must allow them to be "unused." See https://github.com/dart-lang/sdk/issues/39607. +// ignore_for_file: unused_ignore_code import 'dart:async'; import 'package:flutter_tools/src/base/file_system.dart'; diff --git a/packages/flutter_tools/test/general.shard/devfs_test.dart b/packages/flutter_tools/test/general.shard/devfs_test.dart index ca1a6698864..bb1cae37171 100644 --- a/packages/flutter_tools/test/general.shard/devfs_test.dart +++ b/packages/flutter_tools/test/general.shard/devfs_test.dart @@ -4,7 +4,7 @@ import 'dart:async'; import 'dart:convert'; -import 'dart:io'; // ignore: dart_io_import +import 'dart:io'; // ignore_for_flutter: dart_io_import import 'package:file/file.dart'; import 'package:file/memory.dart'; diff --git a/packages/flutter_tools/test/general.shard/flutter_manifest_test.dart b/packages/flutter_tools/test/general.shard/flutter_manifest_test.dart index f4b34ee0cac..8c3b56b273f 100644 --- a/packages/flutter_tools/test/general.shard/flutter_manifest_test.dart +++ b/packages/flutter_tools/test/general.shard/flutter_manifest_test.dart @@ -230,7 +230,7 @@ flutter: expect(fooFontAsset1.style, 'italic'); final Font barFont = fonts[1]; - const String fontDescriptor = '{family: bar, fonts: [{asset: a/baz}, {weight: 400, style: italic, asset: a/baz}]}'; // ignore: always_specify_types + const String fontDescriptor = '{family: bar, fonts: [{asset: a/baz}, {weight: 400, style: italic, asset: a/baz}]}'; expect(barFont.descriptor.toString(), fontDescriptor); expect(barFont.familyName, 'bar'); final List barAssets = barFont.fontAssets; diff --git a/packages/flutter_tools/test/general.shard/forbidden_imports_test.dart b/packages/flutter_tools/test/general.shard/forbidden_imports_test.dart index 047f0ecde61..9318e8602ff 100644 --- a/packages/flutter_tools/test/general.shard/forbidden_imports_test.dart +++ b/packages/flutter_tools/test/general.shard/forbidden_imports_test.dart @@ -52,7 +52,7 @@ void main() { for (File file in files) { for (String line in file.readAsLinesSync()) { if (line.startsWith(RegExp(r'import.*dart:io')) && - !line.contains('ignore: dart_io_import')) { + !line.contains('ignore_for_flutter: dart_io_import')) { final String relativePath = fs.path.relative(file.path, from:flutterTools); fail("$relativePath imports 'dart:io'; import 'lib/src/base/io.dart' instead"); } @@ -99,7 +99,7 @@ void main() { for (File file in files) { for (String line in file.readAsLinesSync()) { if (line.startsWith(RegExp(r'import.*package:path/path.dart')) && - !line.contains('ignore: package_path_import')) { + !line.contains('ignore_for_flutter: package_path_import')) { final String relativePath = fs.path.relative(file.path, from:flutterTools); fail("$relativePath imports 'package:path/path.dart'; use 'fs.path' instead"); } @@ -124,7 +124,7 @@ void main() { for (File file in files) { for (String line in file.readAsLinesSync()) { if (line.startsWith(RegExp(r'import.*dart:convert')) && - !line.contains('ignore: dart_convert_import')) { + !line.contains('ignore_for_flutter: dart_convert_import')) { final String relativePath = fs.path.relative(file.path, from:flutterTools); fail("$relativePath imports 'dart:convert'; import 'lib/src/convert.dart' instead"); } diff --git a/packages/flutter_tools/tool/tool_coverage.dart b/packages/flutter_tools/tool/tool_coverage.dart index 66970646e15..9e7cf5205be 100644 --- a/packages/flutter_tools/tool/tool_coverage.dart +++ b/packages/flutter_tools/tool/tool_coverage.dart @@ -2,6 +2,10 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. +// The "// ignore:" comments found in strings in this file are counted as _actual_ ignore comments. +// Since the "// ignore:" comments found in strings don't actually ignore anythin in _this_ file, +// we must allow them to be "unused." See https://github.com/dart-lang/sdk/issues/39607. +// ignore_for_file: unused_ignore_code import 'dart:async'; import 'dart:developer'; import 'dart:io';