Skip to content

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Dec 4, 2020

Must not be landed until after #71738

Otherwise this is an identical reland. The linked PR fixes a bug in our Android build logic to make this work now.

/cc @blasten @jonahwilliams @Piinks

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Dec 4, 2020
@google-cla google-cla bot added the cla: yes label Dec 4, 2020
print(result.stdout);
print('================================= STDERR =======================================');
print(result.stderr);
fail('flutter drive failed, see output.');
Copy link
Contributor

Choose a reason for hiding this comment

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

This is probably flutter driver* right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, this is referring to the command flutter drive (which is unfortunately easily confused with and definitely related to flutter_driver).

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah so sorry!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No worries, thanks for looking!

@dnfield
Copy link
Contributor Author

dnfield commented Dec 7, 2020

The PR that fixed functionality this depends on has landed.

@dnfield dnfield marked this pull request as ready for review December 7, 2020 17:25
@dnfield dnfield requested a review from blasten December 7, 2020 17:25
Copy link

@blasten blasten 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 Dec 7, 2020

I'm going to file a follow up PR to address updating the default iOS project to support running a test out of the box without flutter drive.

@fluttergithubbot fluttergithubbot merged commit 02eab12 into flutter:master Dec 7, 2020
@dnfield dnfield deleted the integration_template branch December 7, 2020 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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.

4 participants