Unverified Commit e27a19e3 authored by Michael Goderbauer's avatar Michael Goderbauer Committed by GitHub

Remove outdated ignores (#108924)

parent b972f872
...@@ -62,8 +62,6 @@ class MyHomePageState extends State<MyHomePage> { ...@@ -62,8 +62,6 @@ class MyHomePageState extends State<MyHomePage> {
// the placeholder text. // the placeholder text.
Future<void>.delayed(const Duration(milliseconds: 750), () { Future<void>.delayed(const Duration(milliseconds: 750), () {
setState(() { setState(() {
// See https://github.com/dart-lang/sdk/issues/46894
// ignore: prefer_const_constructors
postLoadDisplayWidget = component1.LogoScreen(); postLoadDisplayWidget = component1.LogoScreen();
}); });
}); });
......
...@@ -150,9 +150,8 @@ class StackFrameEquality implements Equality<StackFrame> { ...@@ -150,9 +150,8 @@ class StackFrameEquality implements Equality<StackFrame> {
e1.method == e2.method; e1.method == e2.method;
} }
// TODO(dnfield): This ignore shouldn't be necessary, see https://github.com/dart-lang/sdk/issues/46477
@override @override
int hash(StackFrame e) { // ignore: avoid_renaming_method_parameters int hash(StackFrame e) {
return Object.hash(e.number, e.packageScheme, e.package, e.packagePath, e.line, e.column, e.className, e.method); return Object.hash(e.number, e.packageScheme, e.package, e.packagePath, e.line, e.column, e.className, e.method);
} }
......
...@@ -234,14 +234,11 @@ void main() { ...@@ -234,14 +234,11 @@ void main() {
KeyboardEvent dispatchKeyboardEvent( KeyboardEvent dispatchKeyboardEvent(
EventTarget target, String type, Map<String, dynamic> args) { EventTarget target, String type, Map<String, dynamic> args) {
// ignore: implicit_dynamic_function
final Object jsKeyboardEvent = js_util.getProperty(window, 'KeyboardEvent') as Object; final Object jsKeyboardEvent = js_util.getProperty(window, 'KeyboardEvent') as Object;
final List<dynamic> eventArgs = <dynamic>[ final List<dynamic> eventArgs = <dynamic>[
type, type,
args, args,
]; ];
// ignore: implicit_dynamic_function
final KeyboardEvent event = js_util.callConstructor( final KeyboardEvent event = js_util.callConstructor(
jsKeyboardEvent, js_util.jsify(eventArgs) as List<dynamic>) jsKeyboardEvent, js_util.jsify(eventArgs) as List<dynamic>)
as KeyboardEvent; as KeyboardEvent;
......
...@@ -325,7 +325,7 @@ abstract class TestWidgetsFlutterBinding extends BindingBase ...@@ -325,7 +325,7 @@ abstract class TestWidgetsFlutterBinding extends BindingBase
} }
@override @override
// ignore: MUST_CALL_SUPER // ignore: must_call_super
void initLicenses() { void initLicenses() {
// Do not include any licenses, because we're a test, and the LICENSE file // Do not include any licenses, because we're a test, and the LICENSE file
// doesn't get generated for tests. // doesn't get generated for tests.
......
...@@ -36,7 +36,6 @@ String? _keyLabel(LogicalKeyboardKey key) { ...@@ -36,7 +36,6 @@ String? _keyLabel(LogicalKeyboardKey key) {
return null; return null;
} }
// ignore: avoid_classes_with_only_static_members
/// A class that serves as a namespace for a bunch of keyboard-key generation /// A class that serves as a namespace for a bunch of keyboard-key generation
/// utilities. /// utilities.
class KeyEventSimulator { class KeyEventSimulator {
......
...@@ -100,7 +100,7 @@ class FlutterVersion { ...@@ -100,7 +100,7 @@ class FlutterVersion {
String? _repositoryUrl; String? _repositoryUrl;
String? get repositoryUrl { String? get repositoryUrl {
final String _ = channel; // ignore: no_leading_underscores_for_local_identifiers final String _ = channel;
return _repositoryUrl; return _repositoryUrl;
} }
......
...@@ -547,7 +547,7 @@ void main() { ...@@ -547,7 +547,7 @@ void main() {
'-w', '1', '-w', '1',
'testhostname', 'testhostname',
], ],
postBuildCommand: null, // ignore: avoid_redundant_argument_values postBuildCommand: null,
installCommand: const <String>[ installCommand: const <String>[
'scp', 'scp',
'-r', '-r',
...@@ -637,7 +637,7 @@ void main() { ...@@ -637,7 +637,7 @@ void main() {
'-w', '1', '-w', '1',
'192.168.178.1', '192.168.178.1',
], ],
postBuildCommand: null, // ignore: avoid_redundant_argument_values postBuildCommand: null,
installCommand: const <String>[ installCommand: const <String>[
'scp', 'scp',
'-r', '-r',
...@@ -727,7 +727,7 @@ void main() { ...@@ -727,7 +727,7 @@ void main() {
'-w', '1', '-w', '1',
'::1', '::1',
], ],
postBuildCommand: null, // ignore: avoid_redundant_argument_values postBuildCommand: null,
installCommand: const <String>[ installCommand: const <String>[
'scp', 'scp',
'-r', '-r',
...@@ -821,7 +821,7 @@ void main() { ...@@ -821,7 +821,7 @@ void main() {
'-w', '1', '-w', '1',
'testhostname', 'testhostname',
], ],
postBuildCommand: null, // ignore: avoid_redundant_argument_values postBuildCommand: null,
installCommand: <String>[ installCommand: <String>[
'scp', 'scp',
'-r', '-r',
...@@ -901,7 +901,7 @@ void main() { ...@@ -901,7 +901,7 @@ void main() {
'-w', '1', '-w', '1',
'testhostname', 'testhostname',
], ],
postBuildCommand: null, // ignore: avoid_redundant_argument_values postBuildCommand: null,
installCommand: const <String>[ installCommand: const <String>[
'scp', 'scp',
'-r', '-r',
...@@ -1226,7 +1226,7 @@ void main() { ...@@ -1226,7 +1226,7 @@ void main() {
'testhostname', 'testhostname',
], ],
pingSuccessRegex: RegExp(r'[<=]\d+ms'), pingSuccessRegex: RegExp(r'[<=]\d+ms'),
postBuildCommand: null, // ignore: avoid_redundant_argument_values postBuildCommand: null,
installCommand: const <String>[ installCommand: const <String>[
'scp', 'scp',
'-r', '-r',
......
...@@ -251,7 +251,6 @@ void main() { ...@@ -251,7 +251,6 @@ void main() {
// Unfortunately Device, despite not being immutable, has an `operator ==`. // Unfortunately Device, despite not being immutable, has an `operator ==`.
// Until we fix that, we have to also ignore related lints here. // Until we fix that, we have to also ignore related lints here.
// ignore: avoid_implementing_value_types
class ThrowingScreenshotDevice extends ScreenshotDevice { class ThrowingScreenshotDevice extends ScreenshotDevice {
@override @override
Future<LaunchResult> startApp( Future<LaunchResult> startApp(
......
...@@ -38,7 +38,7 @@ void main() { ...@@ -38,7 +38,7 @@ void main() {
testWithoutContext('AndroidWorkflow handles a null AndroidSDK', () { testWithoutContext('AndroidWorkflow handles a null AndroidSDK', () {
final AndroidWorkflow androidWorkflow = AndroidWorkflow( final AndroidWorkflow androidWorkflow = AndroidWorkflow(
featureFlags: TestFeatureFlags(), featureFlags: TestFeatureFlags(),
androidSdk: null, // ignore: avoid_redundant_argument_values androidSdk: null,
operatingSystemUtils: FakeOperatingSystemUtils(), operatingSystemUtils: FakeOperatingSystemUtils(),
); );
...@@ -391,7 +391,7 @@ Review licenses that have not been accepted (y/N)? ...@@ -391,7 +391,7 @@ Review licenses that have not been accepted (y/N)?
); );
final AndroidValidator androidValidator = AndroidValidator( final AndroidValidator androidValidator = AndroidValidator(
androidStudio: null, // ignore: avoid_redundant_argument_values androidStudio: null,
androidSdk: sdk, androidSdk: sdk,
fileSystem: fileSystem, fileSystem: fileSystem,
logger: logger, logger: logger,
...@@ -439,7 +439,7 @@ Review licenses that have not been accepted (y/N)? ...@@ -439,7 +439,7 @@ Review licenses that have not been accepted (y/N)?
..directory = fileSystem.directory('/foo/bar'); ..directory = fileSystem.directory('/foo/bar');
final AndroidValidator androidValidator = AndroidValidator( final AndroidValidator androidValidator = AndroidValidator(
androidStudio: null, // ignore: avoid_redundant_argument_values androidStudio: null,
androidSdk: sdk, androidSdk: sdk,
fileSystem: fileSystem, fileSystem: fileSystem,
logger: logger, logger: logger,
...@@ -488,7 +488,7 @@ Review licenses that have not been accepted (y/N)? ...@@ -488,7 +488,7 @@ Review licenses that have not been accepted (y/N)?
final ValidationResult validationResult = await AndroidValidator( final ValidationResult validationResult = await AndroidValidator(
androidSdk: sdk, androidSdk: sdk,
androidStudio: null, // ignore: avoid_redundant_argument_values androidStudio: null,
fileSystem: fileSystem, fileSystem: fileSystem,
logger: logger, logger: logger,
platform: FakePlatform()..environment = <String, String>{'HOME': '/home/me', 'JAVA_HOME': 'home/java'}, platform: FakePlatform()..environment = <String, String>{'HOME': '/home/me', 'JAVA_HOME': 'home/java'},
...@@ -510,8 +510,8 @@ Review licenses that have not been accepted (y/N)? ...@@ -510,8 +510,8 @@ Review licenses that have not been accepted (y/N)?
testWithoutContext('Mentions `flutter config --android-sdk if user has no AndroidSdk`', () async { testWithoutContext('Mentions `flutter config --android-sdk if user has no AndroidSdk`', () async {
final ValidationResult validationResult = await AndroidValidator( final ValidationResult validationResult = await AndroidValidator(
androidSdk: null, // ignore: avoid_redundant_argument_values androidSdk: null,
androidStudio: null, // ignore: avoid_redundant_argument_values androidStudio: null,
fileSystem: fileSystem, fileSystem: fileSystem,
logger: logger, logger: logger,
platform: FakePlatform()..environment = <String, String>{'HOME': '/home/me', 'JAVA_HOME': 'home/java'}, platform: FakePlatform()..environment = <String, String>{'HOME': '/home/me', 'JAVA_HOME': 'home/java'},
......
...@@ -566,7 +566,6 @@ void main() { ...@@ -566,7 +566,6 @@ void main() {
outputStdout().join('\n'), outputStdout().join('\n'),
'$message' // initial message '$message' // initial message
'${" " * 4}${" " * 8}' // margin (4) and space for the time at the end (8) '${" " * 4}${" " * 8}' // margin (4) and space for the time at the end (8)
// ignore: missing_whitespace_between_adjacent_strings
'$a' // first tick '$a' // first tick
'$blankLine' // clearing the line '$blankLine' // clearing the line
'Rude Interrupting Cow\n' // message 'Rude Interrupting Cow\n' // message
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be // Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file. // found in the LICENSE file.
import 'dart:io' as io; // ignore: dart_io_import import 'dart:io' as io;
import 'package:flutter_tools/src/base/io.dart'; import 'package:flutter_tools/src/base/io.dart';
import 'package:flutter_tools/src/device_port_forwarder.dart'; import 'package:flutter_tools/src/device_port_forwarder.dart';
import 'package:test/fake.dart'; import 'package:test/fake.dart';
......
...@@ -4,8 +4,6 @@ ...@@ -4,8 +4,6 @@
// @dart = 2.8 // @dart = 2.8
// ignore_for_file: avoid_redundant_argument_values
import 'dart:io' hide Directory, File; import 'dart:io' hide Directory, File;
import 'package:flutter_tools/src/artifacts.dart'; import 'package:flutter_tools/src/artifacts.dart';
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment