Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Conversation

@collinjackson
Copy link
Contributor

@collinjackson collinjackson commented Mar 15, 2019

Fixes flutter/flutter#26080

This teaches Cirrus to run the new drive-examples command added in flutter/plugin_tools#29 on iOS and Android simulators.

I'm sharding the plugins tests more and requesting more powerful Linux VMs to ensure that the tests don't time out now that they're running simulators.

@collinjackson collinjackson requested a review from amirh March 15, 2019 04:30
@collinjackson collinjackson changed the title [shared_preferences] add driver test to cirrus add driver test command to cirrus Mar 15, 2019
@collinjackson collinjackson removed the WIP label Mar 15, 2019
Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

Not sure what are the implications of increasing the resource requirements? (will we run our of cloud credits faster?) worth running this by @gspencergoog

Other than that and the shard count nit LGTM

@collinjackson collinjackson merged commit d3b3d60 into flutter:master Mar 18, 2019
@collinjackson collinjackson deleted the run_driver_tests branch March 18, 2019 20:11
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
This teaches Cirrus to run the new drive-examples command added in flutter/plugin_tools#29 on iOS and Android simulators.

I'm sharding the plugins tests more and requesting more powerful Linux VMs to ensure that the tests don't time out now that they're running simulators.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants