Objectmapper: Hi there are some Error in Xcode Version 10.0 beta (10L176w) & Swift 4.x

Created on 5 Jun 2018  路  12Comments  路  Source: tristanhimmelman/ObjectMapper

like this
screen shot 2018-06-05 at 9 26 57 pm

Most helpful comment

I have created a PR at https://github.com/Hearst-DD/ObjectMapper/pull/978 to hopefully fix this issue. I noticed, after creating it, that another PR exists which has failed tests. Hopefully this produces the right results!

All 12 comments

seeing the same here

I believe that this a duplicate of #975

seeing the same issue

I have created a PR at https://github.com/Hearst-DD/ObjectMapper/pull/978 to hopefully fix this issue. I noticed, after creating it, that another PR exists which has failed tests. Hopefully this produces the right results!

Is there any timeline to be getting a fix in for this? I would like to be moving forward using Xcode 10 to prep for iOS 12, however, this is making it so I cannot even compile our main app. Thanks in advance!

Hi Josh! Feel free to contribute to the PR to help us get closer to a working build. I believe this PR technically works, so you would be able to check the branch out directly into your project.

seeing the same issue

When the update will come.I will transfer if not?

The compile errors are caused by a change in Swift 4.2 described in Abolish ImplicitlyUnwrappedOptional type Swift Evolution.

I have forked the repo and fixed the compile errors, but I don't think it should be merged into master yet. @tristanhimmelman would you accept a PR into master, or could you perhaps create a swift4.2 branch that we can submit PR's against?

There are already three PRs pending for this issue (#976, #978, #980), one of which have passed all tests.

Saw that moments after posting my comment. Great to have options to choose from ;-)

980 solved

Was this page helpful?
0 / 5 - 0 ratings

Related issues

YunyueLin picture YunyueLin  路  3Comments

delbyze picture delbyze  路  3Comments

nearspears picture nearspears  路  4Comments

zhengying picture zhengying  路  4Comments

jperera84 picture jperera84  路  4Comments