Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace git-based MasterSource with CDN-based TrunkSource #552

Merged
merged 7 commits into from
Jul 2, 2019

Conversation

igor-makarov
Copy link
Contributor

@igor-makarov igor-makarov commented Jun 19, 2019

See CocoaPods#8923 for more info.

@igor-makarov igor-makarov force-pushed the cdn-master-source branch 12 times, most recently from b66c786 to 168b682 Compare June 23, 2019 12:31
@igor-makarov igor-makarov changed the title [DNM] Cdn master source Replace git-based MasterSource with CDN-based TrunkSource Jun 24, 2019
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@dnkoutso dnkoutso added this to the 1.8 milestone Jun 27, 2019
Copy link
Contributor

@dnkoutso dnkoutso left a comment

Choose a reason for hiding this comment

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

Please retrieve one more approval for this.

@dnkoutso
Copy link
Contributor

Sorry for the blast of reviewers. I just think one more person should take a look for swapping out the master source!

This is super exciting.

@igor-makarov
Copy link
Contributor Author

Thank you!

Copy link
Member

@amorde amorde left a comment

Choose a reason for hiding this comment

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

Looking great! couple of small questions

spec/source/manager_spec.rb Show resolved Hide resolved
lib/cocoapods-core/source/manager.rb Show resolved Hide resolved
@igor-makarov
Copy link
Contributor Author

Fixed some edge cases in CDN URL handling + added a few tests for the trailing slashes/newlines in CDN URL.

@dnkoutso
Copy link
Contributor

Please rebase now that 1.7.3 is released and ensure CHANGELOG entry is at the right section (master)

@igor-makarov
Copy link
Contributor Author

Rebased Core, CocoaPods and Integration. 💪

@dnkoutso
Copy link
Contributor

dnkoutso commented Jul 2, 2019

@igor-makarov just need a rebase and we can land.

@igor-makarov
Copy link
Contributor Author

Done!

@dnkoutso dnkoutso merged commit 01641a6 into CocoaPods:master Jul 2, 2019
@igor-makarov igor-makarov deleted the cdn-master-source branch July 2, 2019 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants