Skip to content

reland driver vm_service migration#69126

Merged
dnfield merged 2 commits intoflutter:masterfrom
dnfield:reland_driver
Oct 27, 2020
Merged

reland driver vm_service migration#69126
dnfield merged 2 commits intoflutter:masterfrom
dnfield:reland_driver

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Oct 27, 2020

Diff from last time is in 6c086e4

  • Close socket on retries
  • Use pub run test to run the host test file.

@jonahwilliams

See (#68654) (#69074) (#69077) (#69089) for history

@flutter-dashboard flutter-dashboard bot added a: tests "flutter test", flutter_test, or one of our tests framework flutter/packages/flutter repository. See also f: labels. c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels. labels Oct 27, 2020
@google-cla google-cla bot added the cla: yes label Oct 27, 2020
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dnfield
Copy link
Contributor Author

dnfield commented Oct 27, 2020

Failure is fixed on master. Going to land this as is.

@dnfield dnfield merged commit 36f9af3 into flutter:master Oct 27, 2020
@dnfield dnfield deleted the reland_driver branch October 27, 2020 19:38
@dnfield
Copy link
Contributor Author

dnfield commented Oct 27, 2020

This one looks like it's going to stick!

@amirh
Copy link
Contributor

amirh commented Oct 28, 2020

Following this PR integration tests in flutter/plugins are failing with: Could not find package "test". Did you forget to add a dependency?

See: https://cirrus-ci.com/task/4631268350492672?command=build#L255

I'm not sure if it means that the code in flutter/plugins was bad to start with, but we should probably consider this as a breaking change...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests "flutter test", flutter_test, or one of our tests c: contributor-productivity Team-specific productivity, code health, technical debt. framework flutter/packages/flutter repository. See also f: labels. tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants