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

Xcode 9.1 & Swift 4 support #83

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mokagio
Copy link

@mokagio mokagio commented Nov 12, 2017

This PR upgrades the project to be in line with Xcode 9.1 and Swift 4.

Nothing was really necessary.

Note that I couldn't use the migration wizard. It made Xcode 9.1 crash 4
out of 4 times.

What I did was simply bumping the Swift version in the project file, and
ensuring it built all right.
@yzhong52
Copy link

I am wondering what's the plan for this. Because as far as I know, Swift 4 already have native support for JSON encoding and decoding.

@mokagio
Copy link
Author

mokagio commented Mar 22, 2018

@yzhong52 my 2 cents. I use this library in a side project that is live in the Store. I don't have much time to spend on that project, moving to Swift's native Codable has little return of investment for me right now. That's why I forked the library to add Swift 4 support, so that I could still use it in that project without having to spend time migrating stuff that works already.

@yzhong52
Copy link

And looking at your diff, there is no code change at all for Swift 4?

@yzhong52
Copy link

yzhong52 commented Apr 3, 2018

@mokagio also it looks like the build doesn't pass with your changes. https://travis-ci.org/matthewcheok/JSONCodable/builds/300894536

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.

None yet

2 participants