Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

Swift 3.0 Beta Seed 6 #70

Closed
wants to merge 2 commits into from
Closed

Conversation

Bersaelor
Copy link

My team and I were interested in Lyft/mapper as a replacement for ObjectMapper and we needed it to work with swift3 beta 6.

This is just a proposal that makes the tests succeed, but may not be the best solution.
Some method parameters should be adjusted to fit the spirit of the 'Grand renaming' more closely.
For example this method that could be renamed to:
JSON(from field: String) so it would read let result = JSON(from: string)

Furthermore we need to be aware that a 'var some: Any' can now be downcast to 'AnyObject' everytime, like here.

@keith
Copy link
Contributor

keith commented Sep 12, 2016

Hey @Bersaelor thanks for submitting this! We've actually merged #75 and a few other changes to keep it up to date. Please track #76 for progress!

@keith keith closed this Sep 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants