Skip to content

Surport Swift 5#499

Closed
ceeyang wants to merge 1 commit intoEsqarrouth:masterfrom
ceeyang:Swift5.0
Closed

Surport Swift 5#499
ceeyang wants to merge 1 commit intoEsqarrouth:masterfrom
ceeyang:Swift5.0

Conversation

@ceeyang
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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?

//

import Foundation
import AVFoundation
Copy link
Copy Markdown
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 ?

// Copyright (c) 2015 Goktug Yilmaz. All rights reserved.
//

import Foundation
Copy link
Copy Markdown
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
Copy Markdown
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