-
Notifications
You must be signed in to change notification settings - Fork 91
Swift 3 Support #52
Comments
Yeah - definitely got plans to update/maintain it for Swift 3. Need to look through the some of the changes in Foundation though - as ideally would like to improve the underlying decimal number maths to be more Swifty/ValueType. Will have some progress during the week I expect. |
Unfortunately at the moment, this is sort of reliant on Swift 3.0 version of ValueCoding too, which has it's own issues, see: danthorpe/ValueCoding#18. |
There is a pull request, #55 which has Swift 3.0 support for Money. There are a few breaking changes - mostly around Similarly there are number of other Swift 3.0 style name changes. |
Hey @danthorpe sorry for the question but to use the branch of Swift 3 with pods... I've being trying to use it and still pulls the one for 2.3, again sorry, may be something really dumb to do. |
@danthorpe I'm still getting lots of errors when building Swift 3 project with your #55 PR. |
I'm using Money through Cocoapods.
Using the latest What is the recommended workaround for using Money in a Swift 3 app at present? Should I not be using CocoaPods to install Money? |
Hi @msewell - yeah, sorry - known issue r.e. the |
Done! Sorry it took so long to update - real job and ProcedureKit have got in the way! Anyway, can now do this
for CocoaPods, or this:
for Carthage. |
Thanks @danthorpe for your hard work! T("T) |
Just updating a project to swift 3 and using the suggest line above
I get this error on
I assume I've missed something simple, any help would be great |
@danthorpe I am getting the same as @simonbarker |
Hi @danthorpe ! I am using Money 2.0.1 for my Swift 3.2 project using Xcode 9.0 and I getting some errors like this: |
Thanks for your quick help @danthorpe , previous error is missing now. But there is another one: |
Yeah, haven't spent much time with custom currencies at the moment, and Instead, you can try conforming to |
Hi,
Are there any intentions for an update to support Swift 3 for this library? If not I'm happy to take a stab at it and submit a PR if you're open to it.
Thanks for a great library!
The text was updated successfully, but these errors were encountered: