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

WIP Reduce CI testing time, add caching #2357

Closed
wants to merge 35 commits into from
Closed

WIP Reduce CI testing time, add caching #2357

wants to merge 35 commits into from

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Dec 5, 2019 •
edited
Loading

Description

  • CocoaPods 1.8 uses a CDN instead of a local spec repository. pod repo update is no longer necessary. Remove.
  • Add pub cache
  • Add stable Flutter SDK cache

@jmagman
Copy link
Member Author

jmagman commented Dec 5, 2019 •
edited
Loading

Another attempt at #2126

@collinjackson collinjackson self-requested a review December 6, 2019 02:09
@jmagman jmagman changed the title WIP Remove pod repo update from Cirrus script Reduce CI testing time, add caching Dec 6, 2019
@jmagman jmagman changed the title Reduce CI testing time, add caching WIP Reduce CI testing time, add caching Dec 6, 2019
@collinjackson collinjackson self-requested a review December 6, 2019 21:43
collinjackson
collinjackson approved these changes Dec 6, 2019
View reviewed changes
Copy link
Contributor

@collinjackson collinjackson left a comment

Choose a reason for hiding this comment

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

These templates are great -- I didn't know that was available.

I'd love to find a way to share more of this with Flutterfire somehow, but I'm not sure if there's an easy way to do that.

@jmagman
Copy link
Member Author

jmagman commented Dec 6, 2019

These templates are great -- I didn't know that was available.

I'd love to find a way to share more of this with Flutterfire somehow, but I'm not sure if there's an easy way to do that.

Sorry you're getting spammed--the only way to test it to keep pushing changes...

@amirh amirh mentioned this pull request Dec 6, 2019
Merged
@jmagman jmagman closed this Jul 16, 2020
@jmagman jmagman deleted the pod branch June 14, 2022 18:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@collinjackson collinjackson collinjackson approved these changes

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jmagman @collinjackson @googlebot