Skip to content

Conversation

@sburlewapg
Copy link
Contributor

This updates the project to Swift 5, and also updates the iOS simulator to iPhone XS.

@sburlewapg
Copy link
Contributor Author

@stephencelis I don't suppose you'd have time for one more PR merge & version release? 🙏

@karishmapujara
Copy link

@stephencelis please release for Swift 5

@ypopovych
Copy link
Collaborator

Is swift 4.2 version can be compiled with Swift5?

@ypopovych
Copy link
Collaborator

Also, @sburlewapg, can you please add comment into README, that for Swift 4 "0.11" version should be used

- Updated README to clarify Swift 5 vs Swift 4.2 supported versions
- Updated minimum CocoaPods version
- Updated SPM language versions
@sburlewapg
Copy link
Contributor Author

@ypopovych done. Let me know if you prefer any changes to that README note.

@ypopovych
Copy link
Collaborator

Can you also update // swift-tools-version:4.0 to 5.0?

- Updated swift tools version in Package.swift
- Updated "OS X" references to "macOS"
@sburlewapg
Copy link
Contributor Author

@ypopovych done.

@stephencelis could I bother you for a review & merge for this PR once Travis finishes, or is there another person I should reach out to who can do that?

Copy link
Owner

@stephencelis stephencelis left a comment

Choose a reason for hiding this comment

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

@sburlewapg Just a coupla small things!

@stephencelis
Copy link
Owner

@sburlewapg (I think it's also totally fine to not make the exact changes I've suggested, but we should ensure that the SwiftPM tests still pass on CI before merging.)

@stephencelis
Copy link
Owner

Can you also update // swift-tools-version:4.0 to 5.0?

@ypopovych I think we want to keep the tools version as a sort of minimum compiler deployment version.

Co-Authored-By: sburlewapg <49003548+sburlewapg@users.noreply.github.com>
@sburlewapg
Copy link
Contributor Author

@stephencelis I totally agree, and I'm fine with bringing your suggested changes into the PR to make sure that all of the tests are happy.

stephencelis and others added 2 commits April 22, 2019 12:38
Co-Authored-By: sburlewapg <49003548+sburlewapg@users.noreply.github.com>
@sburlewapg
Copy link
Contributor Author

@stephencelis looks like the tests all pass! Good to go whenever you are.

@antileks
Copy link

antileks commented Apr 23, 2019

Tell me, please, is it possible to somehow install the new version using CocoaPods?
The code pod 'SQLite.swift', '~> 0.12' does not work...

@sburlewapg
Copy link
Contributor Author

@antileks this PR has to be merged first, and then somebody needs to release the new pod version.

@ypopovych
Copy link
Collaborator

I think we can merge it.

@ypopovych ypopovych merged commit 41d2e75 into stephencelis:master Apr 24, 2019
@sburlewapg
Copy link
Contributor Author

@ypopovych Thank you! Are you also able to push the new version to CocoaPods?

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.

5 participants