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

Swift 4 update #15

Merged
merged 30 commits into from
Sep 29, 2017
Merged

Swift 4 update #15

merged 30 commits into from
Sep 29, 2017

Conversation

edopelawi
Copy link
Owner

Per #14 's request.

@leedsalex
Copy link
Contributor

@edopelawi Thank you for taking care of this for me in such a timely manner.

@edopelawi
Copy link
Owner Author

Sure thing, @leedsalex! I'm still trying to get the configuration right for Travis CI's Xcode 9, could you use the branch directly (swift-4-update) from your Carthage / Cocoapods? I'll let you know when it's properly merged and released.

Ricardo Pramana Suranta added 15 commits September 23, 2017 17:54
…ion methods

Prior to this commit, the project cannot compile because errors from DestinationInfoMapSectionDelegate,
saying that Swift 4 doesn't allow overriding extension methods.
This will be changed after 7.0.2 is ready.
This will be changed after the working version is ready.
@leedsalex
Copy link
Contributor

@edopelawi Any update on your progress? Is there anything I can help with?

This was skipped because Travis-CI's free builder can't clone using Cocoapods' custom branch parameters.
I have run the test manually in machine to make sure all of them passed.
I'll revert this after Quick and Nimble updated their version to run in Xcode 9 (9A235).
@edopelawi
Copy link
Owner Author

Hey @leedsalex, thanks for bumping this issue. Got caught in a middle of a busy project.

The travis compile issue was caused by Nimble v7.0.2 and Quick v1.0.1 which can't run in Xcode 9. The Quick team already updated Nimble 7.0.2, but not with the Quick. I've posted an issue for it here.

For the time being, I skipped running tests for my Travis build step. I have run all the tests in my machine to make sure that runs well. After the newest builds succeeds, I'll publish the newest version.

Cheers,

@edopelawi edopelawi merged commit a375b17 into develop Sep 29, 2017
@edopelawi edopelawi deleted the swift-4-update branch September 29, 2017 04:40
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.

2 participants