Unverified Commit 32540774 authored by Alexandre Ardhuin's avatar Alexandre Ardhuin Committed by GitHub

enable lint noop_primitive_operations (#83930)

parent 871b0ce3
...@@ -132,6 +132,7 @@ linter: ...@@ -132,6 +132,7 @@ linter:
- no_duplicate_case_values - no_duplicate_case_values
- no_logic_in_create_state - no_logic_in_create_state
# - no_runtimeType_toString # ok in tests; we enable this only in packages/ # - no_runtimeType_toString # ok in tests; we enable this only in packages/
- noop_primitive_operations
- non_constant_identifier_names - non_constant_identifier_names
- null_check_on_nullable_type_parameter - null_check_on_nullable_type_parameter
- null_closures - null_closures
......
...@@ -274,7 +274,7 @@ class MediaQueryData { ...@@ -274,7 +274,7 @@ class MediaQueryData {
/// right: systemGestureInsets.right, /// right: systemGestureInsets.right,
/// ), /// ),
/// child: Slider( /// child: Slider(
/// value: _currentValue.toDouble(), /// value: _currentValue,
/// onChanged: (double newValue) { /// onChanged: (double newValue) {
/// setState(() { /// setState(() {
/// _currentValue = newValue; /// _currentValue = newValue;
......
...@@ -45,7 +45,7 @@ class UwpTool { ...@@ -45,7 +45,7 @@ class UwpTool {
return <String>[]; return <String>[];
} }
final List<String> packageFamilies = <String>[]; final List<String> packageFamilies = <String>[];
for (final String line in result.stdout.toString().split('\n')) { for (final String line in result.stdout.split('\n')) {
final String packageFamily = line.trim(); final String packageFamily = line.trim();
if (packageFamily.isNotEmpty) { if (packageFamily.isNotEmpty) {
packageFamilies.add(packageFamily); packageFamilies.add(packageFamily);
...@@ -82,7 +82,7 @@ class UwpTool { ...@@ -82,7 +82,7 @@ class UwpTool {
return null; return null;
} }
// Read the process ID from stdout. // Read the process ID from stdout.
final int processId = int.tryParse(result.stdout.toString().trim()); final int processId = int.tryParse(result.stdout.trim());
_logger.printTrace('Launched application $packageFamily with process ID $processId'); _logger.printTrace('Launched application $packageFamily with process ID $processId');
return processId; return processId;
} }
......
...@@ -256,7 +256,7 @@ class WindowsUWPDevice extends Device { ...@@ -256,7 +256,7 @@ class WindowsUWPDevice extends Device {
final List<String> dependencyUris = _getDependencyPaths(buildDirectory, 'x64') final List<String> dependencyUris = _getDependencyPaths(buildDirectory, 'x64')
.map((String path) => Uri.file(path).toString()) .map((String path) => Uri.file(path).toString())
.toList(); .toList();
return _uwptool.installApp(packageUri.toString(), dependencyUris); return _uwptool.installApp(packageUri, dependencyUris);
} }
@override @override
......
...@@ -681,7 +681,6 @@ assembleProfile ...@@ -681,7 +681,6 @@ assembleProfile
'\n' '\n'
"Note that your app won't be available to users running Android SDKs below 19.\n" "Note that your app won't be available to users running Android SDKs below 19.\n"
'Alternatively, try to find a version of this plugin that supports these lower versions of the Android SDK.\n' 'Alternatively, try to find a version of this plugin that supports these lower versions of the Android SDK.\n'
''
) )
); );
}, overrides: <Type, Generator>{ }, overrides: <Type, Generator>{
...@@ -719,7 +718,6 @@ assembleProfile ...@@ -719,7 +718,6 @@ assembleProfile
' checkReleaseBuilds false\n' ' checkReleaseBuilds false\n'
' }\n' ' }\n'
'}\n' '}\n'
''
) )
); );
}, overrides: <Type, Generator>{ }, overrides: <Type, Generator>{
...@@ -757,7 +755,6 @@ Execution failed for task ':app:generateDebugFeatureTransitiveDeps'. ...@@ -757,7 +755,6 @@ Execution failed for task ':app:generateDebugFeatureTransitiveDeps'.
'To regenerate the lockfiles run: `./gradlew :generateLockfiles` in /android/build.gradle\n' 'To regenerate the lockfiles run: `./gradlew :generateLockfiles` in /android/build.gradle\n'
'To remove dependency locking, remove the `dependencyLocking` from /android/build.gradle\n' 'To remove dependency locking, remove the `dependencyLocking` from /android/build.gradle\n'
'\n' '\n'
''
) )
); );
}, overrides: <Type, Generator>{ }, overrides: <Type, Generator>{
......
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