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

Conversation

@jtreanor
Copy link
Contributor

@jtreanor jtreanor commented Oct 9, 2019

We're coming closer to hitting our CircleCI limit each month so I'm making a few small changes to improve usage and hopefully improve the development cycle as well.

In 1.7.0, CocoaPods introduced a CDN specs repo which could be used to replace the huge, slow git-based specs repo. In 1.8.0 CocoaPods started using this by default, so we will need to switch at some stage. I'm starting on this repo as it hopefully is small and manageable.

The only disadvantage of using the CDN specs repo for us is that there is a delay of about 4 minutes from when pod trunk push completes and a pod is available in the CDN specs repo. For WordPressAuthenticator, this means if you try to use a new version of WordPressShared within 4 minutes of it being published, pod install won't find it. You can see deploys to the CocoaPods Specs CDN here. Given this speeds things up by much more than 4 minutes, I feel that this tradeoff is worthwhile.

So here are the changes I made to implement this:

I have also tested the Publish to Trunk CircleCI job here. Run time for that is reduced from ~13 minutes to ~3.5 minutes.

To Test

  • See that CircleCI is green and runs much more quickly.
  • Checkout this branch and run bundle install && bundle exec pod install. Confirm it completes and there are no changes to Podfile.lock.

Copy link
Contributor

@loremattei loremattei left a comment

Choose a reason for hiding this comment

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

This looks great!
Changes make sense to me and I tested as described and it worked as expected.

I wonder if we can add a line about the small delay in availability to the message we post on slack at the end of the pod trunk action.

:shipit:

@jtreanor
Copy link
Contributor Author

jtreanor commented Oct 9, 2019

I wonder if we can add a line about the small delay in availability to the message we post on slack at the end of the pod trunk action.

Thats a very good idea! I can add that.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants