-
Notifications
You must be signed in to change notification settings - Fork 350
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
Introduce Source::Metadata #316
Conversation
end | ||
|
||
def path_fragment(pod_name, version = nil) | ||
prefixes = if prefix_lengths.empty? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this would be a little cleaner if the if/else were a separate method?
|
||
# @return [Array<Source>] The list of the sources with the given names. | ||
# | ||
# @param [Array<#to_s>] names |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this correct? [Array<#to_s>]
- don't think I've seen it before :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah -- it means an array of stuff that responds to to_s
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, an informal protocol - got it
LGTM 👍 |
This also allows for specs directory sharding
762fc36
to
b6df430
Compare
This is ready to merge |
In doing so, we've migrated
Pod::SourcesManager
from CocoaPods/CocoaPods as a singleton to be a proper class, and also added support for sharded specs directories.Fixes CocoaPods/CocoaPods#5062.