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

Adding Podspec for Swift 4 support #158

Merged
merged 3 commits into from
Oct 2, 2017
Merged

Adding Podspec for Swift 4 support #158

merged 3 commits into from
Oct 2, 2017

Conversation

mergesort
Copy link
Contributor

This is create a podspec for the Swift 4 support that was added yesterday.

This is to incorporate the Swift 4 support that was merged into master.
@RuiAAPeres
Copy link
Collaborator

Unfortunately, there was an an API breaking change, so it would be 5.0.0. We shouldn't have actually allowed that change, since AFAIK it's orthogonal to the PR's intent.

In any case, thanks for the PR. Please fix it and I will cut a new release.

Since the Swift 4 upgrade was a breaking change, we're bumping this to 5.0.0 instead of 4.2.0.
@mergesort
Copy link
Contributor Author

Updated.

@zfoltin
Copy link
Collaborator

zfoltin commented Sep 30, 2017

I think it's perfectly fine to have a major version bump when adding support to a new Swift version.
Should probably update the README though...

@mergesort
Copy link
Contributor Author

@zfoltin Updated the README as well.

@zfoltin
Copy link
Collaborator

zfoltin commented Oct 2, 2017

Thanks @mergesort 👍
@RuiAAPeres if you are happy, let's merge this and push to cocoapods, shall we?

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.

3 participants