Skip to content

Update cptr1.podspec - #2

Open
IvanDurak wants to merge 4 commits into
masterfrom
ivan
Open

Update cptr1.podspec#2
IvanDurak wants to merge 4 commits into
masterfrom
ivan

Conversation

@IvanDurak

Copy link
Copy Markdown
Contributor

Updated the version to 8.0

@deniskr

deniskr commented May 11, 2016

Copy link
Copy Markdown
Owner

I have some problems with your code, please see comments

Comment thread cptr1.podspec Outdated
# s.social_media_url = 'https://twitter.com/<TWITTER_USERNAME>'

s.platform = :ios, '7.0'
s.platform = :ios, '8.0'

@deniskr deniskr May 11, 2016

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why not 8.1.a?

Comment thread cptr1.podspec Outdated
# s.social_media_url = 'https://twitter.com/<TWITTER_USERNAME>'

s.platform = :ios, '8.0'
s.platform = :ios, '8.1.a'

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is still not enough. It should be 9.0

@deniskr

deniskr commented May 11, 2016

Copy link
Copy Markdown
Owner

Please address all comments in code

@IvanDurak

Copy link
Copy Markdown
Contributor Author

Did that, check again.
placehold48x55

@deniskr deniskr closed this May 11, 2016
@deniskr deniskr reopened this May 11, 2016
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.

2 participants