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

Surport Swift 5 #499

Closed
wants to merge 1 commit into from
Closed

Surport Swift 5 #499

wants to merge 1 commit into from

Conversation

ceeyang
Copy link

@ceeyang ceeyang commented Apr 18, 2019

Checklist

  • New Extension
  • New Test
  • Changed more than one extension, but all changes are related
  • Trivial change (doesn't require changelog)

@Esqarrouth Esqarrouth requested a review from Khalian April 18, 2019 14:50
Copy link
Collaborator

@Khalian Khalian left a comment

Choose a reason for hiding this comment

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

The changes look relatively straightforward and minor. However the build fails because of https://travis-ci.org/goktugyil/EZSwiftExtensions. Xcode 10.0 cannot be installed on macOS 10.12.

What version of XCode and macOS X are you using to locally build them? We need to match that configuration with the travis.yml file on.

}
}

// public func timePassed() -> TimePassed {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why are we commenting this out?

@@ -7,6 +7,7 @@
//

import Foundation
import AVFoundation
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do we need this ?

@@ -6,6 +6,8 @@
// Copyright (c) 2015 Goktug Yilmaz. All rights reserved.
//

import Foundation
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need this ?

@Khalian
Copy link
Collaborator

Khalian commented May 12, 2019

I have created a remote branch ceeyang-Swift5.0, I will keep pushing in changes to fix builds and stuff, but I would need your help since I am out of date on all the XCode versions and MacOSX versions for latest swift stuff. Regardless, I am attempting a few expts here : #501. This is by no means the official PR for 5.0, this is to facilitate build fixes only.

@Khalian Khalian mentioned this pull request May 12, 2019
4 tasks
@ceeyang ceeyang closed this May 17, 2019
@ceeyang ceeyang reopened this May 17, 2019
@ceeyang ceeyang closed this May 17, 2019
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