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-276: Swift 5 support #233
Conversation
e41f2e2
to
e48666a
Compare
0f655e8
to
40afed7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! great work getting this fixed. It's been that thing looming over us for a while but you've handled it quite nicely 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one nit, otherwise lgtm!! woohoo! thanks so much for figuring all this out 🙂
fatalError("Failed to initialize an iterator over document \(self)") | ||
} | ||
|
||
for pos in 0..<position + 1 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: maybe clearer to just write this as for pos in 0...position
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh yeah duh, silly copy pasta on my part there!
This adds support for compiler(>=5.0)
JSONSerialization changed how they serialized JSON between v4 and v5, so we'll use libbson for JSON cleaning for now.
40afed7
to
e28a2af
Compare
e28a2af
to
036e76e
Compare
That's great, amazing job 👍🏻 |
@valeriomazzeo yup, within the next few days 🙂 |
This introduces full support for Swift 5 on OSX and Linux.