Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove JSONCodable protocol #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

emilwojtaszek
Copy link

@emilwojtaszek emilwojtaszek commented Aug 16, 2017

I've bumped into a problem while using xcode9.
Error I was getting was:

'JSONDecoder' is ambiguous for type lookup in this context

The only way to fix this is to put class in proper namespace by add prefix JSONCodable like JSONCodable.JSONDecoder, but there is a problem that compiler want to use protocol called the same, not lib name

the only idea I've figure out is to remove this protocol

@pronebird
Copy link

I'd really like that patch if it works.

@valerianb
Copy link

In the file that produces that error, you can also do
import class JSONCodable.JSONDecoder
It disambiguates what you are referencing for Xcode !

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

Successfully merging this pull request may close these issues.

None yet

3 participants