-
Notifications
You must be signed in to change notification settings - Fork 27.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[flutter_tools] remove globals from FlutterValidator, add documentation and move tests to new file #67234
[flutter_tools] remove globals from FlutterValidator, add documentation and move tests to new file #67234
Conversation
…r_version_validator
…liams/flutter into split_flutter_version_validator
@@ -28,16 +28,6 @@ class DoctorCommand extends FlutterCommand { | |||
@override | |||
final String description = 'Show information about the installed tooling.'; | |||
|
|||
@override | |||
Future<Set<DevelopmentArtifact>> get requiredArtifacts async { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment below on Flutter validator
buf.writeln(userMessages.flutterBinariesDoNotRun); | ||
if (globals.platform.isLinux) { | ||
buf.writeln(userMessages.flutterBinariesLinuxRepairCommands); | ||
// If the binaries are not downloaded (because android is not enabled), then do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #67352 for the issue where we fix + remove this.
Do not need to worry about the binaries being up to date, because gen_snapshot is a native binary and does not need a matching Dart SDK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Description
Remove globals from the flutter validator class, and refactor the tests into a separate file. Applies some other cleanup like adding doc comments, and making the doctor validator work like it is documented to work - removing the gen_snapshot check if the artifact is not downloaded instead of downloading all android artifacts.
#47161