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

[Podfile::DSL] Allow link_with targets to be passed as the arg list instead of an array #30

Merged
merged 1 commit into from
Oct 11, 2013

Conversation

sharplet
Copy link

Seemed like it would be a nice API not to be required to wrap the target
list in an array explicitly. Hope you like it!

…nstead of an array

Also adds spec coverage to ensure backwards compatibility with the
array style.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e3c93c6 on sharplet:link_with into 97632f1 on CocoaPods:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.18%) when pulling e3c93c6 on sharplet:link_with into 97632f1 on CocoaPods:master.

@fabiopelosin
Copy link
Member

Looks good to me, can you add an entry to the CocoaPods/CocoaPods Changelog crediting yourself?

@sharplet
Copy link
Author

Done.

@fabiopelosin
Copy link
Member

Awesome thanks!

fabiopelosin added a commit that referenced this pull request Oct 11, 2013
[Podfile::DSL] Allow link_with targets to be passed as the arg list instead of an array
@fabiopelosin fabiopelosin merged commit 2f3fb92 into CocoaPods:master Oct 11, 2013
fabiopelosin added a commit to CocoaPods/CocoaPods that referenced this pull request Oct 11, 2013
@orta
Copy link
Member

orta commented Oct 11, 2013

Is the old syntax for this included in pod create XXlib ?

@fabiopelosin
Copy link
Member

No, we don't link with multiple targets there.

@sharplet sharplet deleted the link_with branch January 24, 2014 05:34
Ashton-W pushed a commit to Ashton-W/Core that referenced this pull request Nov 2, 2015
[Podfile::DSL] Allow link_with targets to be passed as the arg list instead of an array
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.

4 participants