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

[ci] Switch to macOS VM with Flutter pre-installed #1313

Merged
merged 4 commits into from
Jul 12, 2019
Merged

[ci] Switch to macOS VM with Flutter pre-installed #1313

merged 4 commits into from
Jul 12, 2019

Conversation

fkorotkov
Copy link
Contributor

No description provided.

@amirh
Copy link
Contributor

amirh commented Jul 11, 2019

@fkorotkov I'm missing context here - do we still need this?

@fkorotkov
Copy link
Contributor Author

@amirh totally forgot about the PR. Such change can help to reduce the 9 minute setup script. I can rebase the PR to see how much faster it is now. 🤔

@amirh
Copy link
Contributor

amirh commented Jul 11, 2019

sounds good, thanks!

@fkorotkov
Copy link
Contributor Author

@amirh rebased and it strips of 3+ minutes from the macOS shard build making it almost 2X faster. 🙌

@amirh
Copy link
Contributor

amirh commented Jul 12, 2019

Great ! 🎉
Good to merge?

@amirh amirh marked this pull request as ready for review July 12, 2019 16:37
@fkorotkov
Copy link
Contributor Author

@amirh yes! 🎉

@amirh amirh merged commit ff8e8ca into flutter:master Jul 12, 2019
@fkorotkov fkorotkov deleted the patch-6 branch July 12, 2019 17:30
amirh added a commit that referenced this pull request Jul 16, 2019
mithun-mondal pushed a commit to bKash-developer/archived_plugins that referenced this pull request Aug 6, 2019
[ci] Switch to macOS VM with Flutter pre-installed
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
[ci] Switch to macOS VM with Flutter pre-installed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants