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

[Specification::DSL] Add prepare_command #1252

Merged
merged 3 commits into from
Aug 2, 2013

Conversation

fabiopelosin
Copy link
Member

@alloy & @orta

Closes #1274

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 4048558 on feature-prepare_command into a0160e7 on master.

@alloy
Copy link
Member

alloy commented Aug 2, 2013

:shipit:

fabiopelosin added a commit that referenced this pull request Aug 2, 2013
[Specification::DSL] Add prepare_command
@fabiopelosin fabiopelosin merged commit c7b274f into master Aug 2, 2013
@fabiopelosin fabiopelosin deleted the feature-prepare_command branch August 2, 2013 08:46
@alloy
Copy link
Member

alloy commented Aug 2, 2013

The implementation looks good to me. Have you by any chance converted some specs to use the prepare_command instead of a pre_install hook?

@fabiopelosin
Copy link
Member Author

I have added it to a spec to touch a header, but nothing more functional.

jzapater pushed a commit to jzapater/CocoaPods that referenced this pull request Sep 17, 2013
Update ViewUtils to version 1.1
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.

Pod spec lint allow comments
3 participants