-
Notifications
You must be signed in to change notification settings - Fork 88
Update for Swift 4.1 and Xcode 9.3 #136
Conversation
@@ -110,7 +110,7 @@ public struct Mapper { | |||
} | |||
|
|||
let rawValues = try array.map { try T.RawValue.fromMap($0) } | |||
return rawValues.flatMap { T(rawValue: $0) ?? defaultValue } | |||
return rawValues.compactMap { T(rawValue: $0) ?? defaultValue } |
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.
It won't build now for anyone that want to use this library with Swift 4.0.X, right?
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.
Yep, I'll wait for the final Xcode 9.3 release and once I merge this I will bump the major version.
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.
Yeah, I guess that's reasonable 👍
@keith Any chance this can be merged soon ?. Xcode 9.3 was released and this is failing on Carthage build. |
@Vkt0r 8.0.0 is tagged with this change. |
@keith 👍 Thanks for quick response. |
No description provided.