-
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] update device discovery constructors #61640
[flutter_tools] update device discovery constructors #61640
Conversation
@@ -26,6 +26,8 @@ void main() { | |||
featureFlags: TestFeatureFlags(), | |||
), | |||
processManager: FakeProcessManager.list(<FakeCommand>[]), | |||
fileSystem: MemoryFileSystem.test(), | |||
platform: FakePlaatform(), |
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.
FakePlaatform
has a typoo
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.
Thaanks!
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
google3_doctor.dart is sad:
|
Fixing google3 here: cl/322401679 |
google3 testing either passed or failed to run :) |
google3 callsites are updated, remove global lookup from constructor parameters.
google3 callsites are updated, remove global lookup from constructor parameters.
Description
google3 callsites are updated, remove global lookup from constructor parameters.