Skip to content
This repository has been archived by the owner on Jul 2, 2018. It is now read-only.

Swift 3.2/4.0 support #79

Open
AtharvaVaidya opened this issue Jun 6, 2017 · 23 comments
Open

Swift 3.2/4.0 support #79

AtharvaVaidya opened this issue Jun 6, 2017 · 23 comments

Comments

@AtharvaVaidya
Copy link

Are there plans to update this library to Swift 3.2 or 4.0?

@danthorpe
Copy link
Owner

Hi @AtharvaVaidya - it is certainly my desire to maintain the project for Swift 3.2 & Swift 4.0. I will hopefully get to Swift 3.2 in the next week or so.

On a larger point, I'm mulling over potential changes for a v3 of the library which would accommodate initialisation from string values such as "USD 10" or "10USD" - which is currently not possible.

@hdoria
Copy link

hdoria commented Jun 9, 2017

+1

@AtharvaVaidya
Copy link
Author

The string initialization would be a very welcome addition. I'd be glad to help implement it.

@mattcorey
Copy link

Swift 3.2/4.0 compatibility seems to be a more basic need - any focus on this would be appreciated!

@lmcd
Copy link

lmcd commented Sep 13, 2017

👍
Looking to migrate a large project.

@maximkhatskevich
Copy link

@danthorpe Were you able to add Swift 4 support or when do you plan to do so? Thanks!

@lmcd
Copy link

lmcd commented Sep 15, 2017

If I can help in any way, let me know. The build errors I'm seeing seem to relate to the use of NSDecimalNumber and lack of conformance to the Numeric protocol.

We could probably resolve these issues by migrating to the Decimal structure, which provides Numeric conformance out of the box.

@danthorpe
Copy link
Owner

Thanks all, working on Swift 4 now.

@uzumakilord
Copy link

@danthorpe Do you solve this problem with the numeric protocol? Where can I find it? I need to migrate a project. Thanks!

@attheodo
Copy link

@danthorpe any ETA on when a new version with Swift3.2/4-XCode9.0 will be out? We've kinda reach the point where Money is breaking our build now :(

@danthorpe
Copy link
Owner

@attheodo I have most of it done, will have a workable branch published tonight which will unblock you. But probably won't have a new version in CocoaPods until Sunday.

@attheodo
Copy link

@danthorpe thanks man. That will help a bunch!

@lmcd
Copy link

lmcd commented Sep 20, 2017

Thanks @danthorpe

@clintonmedbery
Copy link

@danthorpe
Copy link
Owner

I've published the branch: feature/MNY-79_updates_for_xcode_9 which should hopefully unblock everyone from adopting Xcode 9. See: #84

The primary thing which is missing at the moment is full support for localised formatting, ran out of time today, will do this on Sunday. This will get released as a major version 3 as soon as possible. Please chime in on #84 with comments of breaking changes or things which you need.

@OnurVar
Copy link

OnurVar commented Oct 12, 2017

Any updates ?

@sashakid
Copy link

sashakid commented Nov 8, 2017

feature/MNY-79_updates_for_xcode_9

How to perform arithmetic operations in this branch? Dividing is not working anymore:

let sum = Money(floatLiteral: sum)
return "\(sum.dividing(by: PriceService.shared.price).decimal)"

Value of type 'Money' has no member 'dividing'

@clintonmedbery
Copy link

Is this dead now? Haven't seen any updates despite there being progress made.

@clintonmedbery
Copy link

RIP

@Ewg777
Copy link

Ewg777 commented Jan 9, 2018

@danthorpe any updates?

@namchuai
Copy link

well, here we are..

@marbetschar
Copy link

@danthorpe .... anything?

Your Money library looks awesome and it does exactly what I need in my project. But I'm a bit frightened using it, because there is a lot of silence around it for the last year or so. So it seems like it is dead - and coding my own solution, and therefore ignore all of your achievements simply feels not right.

That said, I'll happily provide help in coding to bring this awesome library back to life. So let's start rolling with Swift 4 support!

Where to start best?

@marbetschar
Copy link

@danthorpe I've found that the branch feature/MNY-79_updates_for_xcode_9 is working quite well in Swift v4.

Based upon that, I've created a pull request for FX which adds support for Swift v4, and another one which adds missing functionality in Money to work properly with the new FX.

Based upon this, it would be awesome to create new releases for both frameworks and take the next steps from there.

Any chance we can get that done somewhen soon?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests