Skip to content

Tolerate Swift 5.1#44

Merged
djones6 merged 4 commits intomasterfrom
swift51
Jul 16, 2019
Merged

Tolerate Swift 5.1#44
djones6 merged 4 commits intomasterfrom
swift51

Conversation

@djones6
Copy link
Copy Markdown
Member

@djones6 djones6 commented Jul 16, 2019

Fix format changes in tests for Swift 4.2 and later (todo: are these actually the right results?)
Disable part of a test that assumes dictionary ordering in Swift 5 and later
Build in Swift 5 compiler mode
Fix compilation failure by commenting out an apparently ambiguous < operator on NSNumber with Swift 5.1 (todo: why is this necessary?)

@djones6 djones6 requested a review from ianpartridge July 16, 2019 13:39
@djones6
Copy link
Copy Markdown
Member Author

djones6 commented Jul 16, 2019

So we have no Travis support for Swift 5.1 on macOS at the moment, but I have manually tested that this works on the latest Xcode11 and Catalina beta 3.

@djones6 djones6 merged commit f9b4754 into master Jul 16, 2019
@djones6 djones6 deleted the swift51 branch July 16, 2019 13:50
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