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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support SwiftPM (4.1.0) #40

Merged
merged 1 commit into from Jan 31, 2019
Merged

Support SwiftPM (4.1.0) #40

merged 1 commit into from Jan 31, 2019

Conversation

Nonchalant
Copy link
Contributor

Swift Version of MirrorDiffKit is recognized as Swift 3.3 when using by SwiftPM.

I understand Syntax of MirrorDiffKit is Swift 4.1, but SwiftPM doesn't recognize correctly 馃槺

This issue can be resolved by fixing Package.swift to syntax of SwiftPM (4.1.0) 馃殌

@Kuniwak
Copy link
Owner

Kuniwak commented Sep 25, 2018

@Nonchalant Thanks for reporting.

I understand Syntax of MirrorDiffKit is Swift 4.1

I think MirrorDiffKit still can compile on Swift 3.1 (but I confirmed that compiling tests are failed).

So, we should support both Swift 3.1 and Swift 4.1 to make compatibility with Swift 3.x if we can.
Did you test this patch on Swift 3.1? (and I did not confirmed yet)

@Nonchalant
Copy link
Contributor Author

@Kuniwak I confirmed test is successful on Swift 3.3. Is it OK?

$ swift package --version
// Apple Swift Package Manager - Swift 4.1.0 (swiftpm-14050)

$ swift package generate xcodeproj
// Set swift 3.3 manualy
$ swift test // Successful

2018-09-25 17 37 28

@Kuniwak
Copy link
Owner

Kuniwak commented Sep 29, 2018

@Nonchalant Hmm... This result seems strange, because some tests are not compatible for Swift 3.

I'm gonna investigate the cause.

@Kuniwak Kuniwak changed the base branch from master to dev-4.0.0 January 31, 2019 16:49
@Kuniwak Kuniwak self-requested a review January 31, 2019 16:49
@Kuniwak Kuniwak merged commit dc75dea into Kuniwak:dev-4.0.0 Jan 31, 2019
@Kuniwak
Copy link
Owner

Kuniwak commented Jan 31, 2019

I'm sorry about my tooooooo late reply.

I decided to merge it, because If any problems are occured, we can fix it.
Thanks for your contributions!

@Kuniwak Kuniwak mentioned this pull request Jan 31, 2019
Kuniwak added a commit that referenced this pull request Jan 31, 2019
* Support Swift 5 (#41)

* Update .gitignore

* Update CocoaPods

* Migrate to Swift5

* Update README.md

* Make CircleCI happy by migrating to Swift 4.2.1

* Try to fix the error from "pod spec lint"

Related: CocoaPods/fourflusher#15

* Fix swift_version (#40)

* Ready to be v4.0.0
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.

None yet

2 participants