Skip to content

Commit bb1dea7

Browse files
authored
change messaging around Windows (flutter#8549)
1 parent e4f586d commit bb1dea7

File tree

2 files changed

+7
-9
lines changed

2 files changed

+7
-9
lines changed

packages/flutter_tools/lib/src/commands/test.dart

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,13 @@ class TestCommand extends FlutterCommand {
156156

157157
@override
158158
Future<Null> runCommand() async {
159+
if (platform.isWindows) {
160+
throwToolExit(
161+
'The test command is currently not supported on Windows: '
162+
'https://github.com/flutter/flutter/issues/8516'
163+
);
164+
}
165+
159166
List<String> testArgs = <String>[];
160167

161168
commandValidator();

packages/flutter_tools/lib/src/doctor.dart

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -224,15 +224,6 @@ class _FlutterValidator extends DoctorValidator {
224224
messages.add(new ValidationMessage('Engine revision ${version.engineRevisionShort}'));
225225
messages.add(new ValidationMessage('Tools Dart version ${version.dartSdkVersion}'));
226226

227-
if (platform.isWindows) {
228-
valid = ValidationType.missing;
229-
230-
messages.add(new ValidationMessage.error(
231-
'Flutter tools are not (yet) supported on Windows: '
232-
'https://github.com/flutter/flutter/issues/138.'
233-
));
234-
}
235-
236227
return new ValidationResult(valid, messages,
237228
statusInfo: 'on ${osName()}, channel ${version.channel}');
238229
}

0 commit comments

Comments
 (0)