Skip to content
This repository was archived by the owner on Apr 15, 2020. It is now read-only.

Update to Swift 5#131

Merged
jessesquires merged 7 commits into
jessesquires:xcode11-swift5-1from
kevnm67:feature/swift-5
Sep 23, 2019
Merged

Update to Swift 5#131
jessesquires merged 7 commits into
jessesquires:xcode11-swift5-1from
kevnm67:feature/swift-5

Conversation

@kevnm67
Copy link
Copy Markdown
Collaborator

@kevnm67 kevnm67 commented Sep 5, 2019

Pull request checklist

This fixes issue

Closes issue #117

What's in this pull request?

  • Migrates project settings and code for swift 5 compatibility

@kevnm67 kevnm67 added this to the 8.1.0 milestone Sep 5, 2019
@kevnm67 kevnm67 requested a review from jessesquires September 5, 2019 09:03
@jessesquires
Copy link
Copy Markdown
Owner

Thanks so much @kevnm67 !

Super sorry for the delay here. I'll get to this tomorrow. 😄

kevnm67 and others added 5 commits September 20, 2019 17:56
[Fixed]
- Travis build failure due to incorrect simulator for xcode 10.2 image.
- Updated gemfile to use latest cocoapods [fixing pod lint error](CocoaPods/CocoaPods#8875).
@jessesquires jessesquires changed the title Updated to swift 5 Update to Swift 5 Sep 23, 2019
@jessesquires jessesquires changed the base branch from develop to xcode11-swift5-1 September 23, 2019 22:16
@jessesquires
Copy link
Copy Markdown
Owner

Reverted some of the unrelated changes that I'll take care of on develop instead. 😄

@jessesquires jessesquires merged commit a4172bd into jessesquires:xcode11-swift5-1 Sep 23, 2019
@kevnm67
Copy link
Copy Markdown
Collaborator Author

kevnm67 commented Sep 23, 2019

@jessesquires I would have happily broke out the CI updates 😆 Only reason I added them was the need to fix Travis build failures. I actually have an Xcode 11 PR waiting 😉

@kevnm67 kevnm67 deleted the feature/swift-5 branch September 23, 2019 22:18
@jessesquires
Copy link
Copy Markdown
Owner

@kevnm67 oops! Sorry 😅 I should have asked first

@jessesquires
Copy link
Copy Markdown
Owner

the travis config was also set to Xcode10.2, so a bit out-dated.

merged into a new branch xcode11-swift5-1 and will be pushing updates there now 😄

@kevnm67
Copy link
Copy Markdown
Collaborator Author

kevnm67 commented Sep 23, 2019

@jessesquires All good. Im all for the divide and conquer approach 😉

jessesquires added a commit that referenced this pull request Sep 24, 2019
* Update to Swift 5 (#131)
* Xcode 11 update, settings/config
* Update @unknown default: enum cases
* [SwiftLint] update 0.35, add new rules
* [travis ci] update for xcode 11, add ui tests
* Update README
* [Gems] Update cocoapods 1.7.5, danger 6.0
* update podspec
* update CHANGELOG
* Fix, cleanup, refactor UI Tests
* bump version to 8.1.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants