search

dillidon / alerts-and-pickers

Advanced usage of UIAlertController and pickers based on it: Telegram, Contacts, Location, PhotoLibrary, Country, Phone Code, Currency, Date...
MIT License
5.65k stars 700 forks source link

Migrated to Swift 5 #105

Closed Andrew11US closed 1 year ago

Andrew11US commented 4 years ago

Migrated project to Swift 5

Andrew11US commented 4 years ago

Hey, I have reviewed your pull request. The code seems fine to me except for one thing. I was curious as to why did you replace flatMap with compactMap?

The only reason is that flatMap was deprecated in Xcode 9.3 an up due to misuse and Swift 5 migration tool automatically changes flatMap to compactMap