Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Update macOS native code to Swift 4.0 #1951

Merged
merged 1 commit into from
Sep 11, 2019
Merged

Conversation

yknl
Copy link
Collaborator

@yknl yknl commented Aug 30, 2019

Update macOS native code to Swift 4.0 so that new builds can be made with xcode 10.1+, since Apple has deprecated support for Swift 3.0 in newer xcode versions. PR changes Swift compiler version to 4.0 and updates deprecated syntax to Swift 4.0.

@yknl yknl requested a review from hstove August 30, 2019 19:23
Copy link
Collaborator

@hstove hstove left a comment

Choose a reason for hiding this comment

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

LGTM - I ran the local build script and got the native app with no errors. Seems to be good. Please advise if there is more testing I should be doing.

I'm curious how you came to do this - were you blocked by needing to upgrade? Just good practice? I see that Swift 5 is out and has an auto-upgrade process in Xcode.

@yknl
Copy link
Collaborator Author

yknl commented Sep 7, 2019

Yes, the upgrade is blocking a release since the latest version of xcode deprecated support for swift 3.0.

@yknl yknl merged commit 29b98d3 into develop Sep 11, 2019
@hstove hstove mentioned this pull request Nov 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants