-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
Added ParseKit podspec #125
Conversation
Note: lint fails due to https://github.com/CocoaPods/Specs/issues/110
@@ -0,0 +1,39 @@ | |||
Pod::Spec.new do |s| | |||
s.name = 'ParseKit' | |||
s.version = '1.0' |
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.
Specs without version tags should be added as 0.0.1
changed version to 0.0.1 |
Good. What about asking the author to add tags? |
Wow, you even have a template for this! Thanks. The template assumes that podspec should be in the Specs repo. Should I wait for merge before filing the issue? |
Ok, I think I'll give a link to this pull request. |
Here it is: itod/parsekit#12 |
Thank you. Spec merged, and you have push access to this repo now. |
@siuying, why did you rolled back this commit? |
@SevInf I'm sorry i made a mistake on this... I'm not intended to overwrite the commit |
@SevInf Could you roll it back? |
Done. @siuying, actually you should never use |
@SevInf thanks. understood and noted carefully. |
Homepage: http://parsekit.com/
GH: https://github.com/itod/parsekit
Note:
pod spec lint
fails due to https://github.com/cocoapods/specs/issues/110, but passes if the condition is commented out. I've also smoke-tested it with my iOS project, and it seem to work.