Skip to content
This repository has been archived by the owner on Feb 8, 2021. It is now read-only.

Replace force unwrapping with failable initializer #38

Merged
merged 9 commits into from
Aug 22, 2015
Merged

Replace force unwrapping with failable initializer #38

merged 9 commits into from
Aug 22, 2015

Conversation

hkellaway
Copy link
Owner

Replaces PR #31

Original description from @morganchen12:

Better supporting the unhappy path of failure by producing a nil model object instead of a non-nil model object with all of its fields set to nil. This also allows for non-optional model fields without the risk of runtime errors.

@hkellaway hkellaway mentioned this pull request Aug 22, 2015
Closed
@hkellaway hkellaway added this to the 0.5.0 milestone Aug 22, 2015
hkellaway pushed a commit that referenced this pull request Aug 22, 2015
Merge pull request #38 from hkellaway/feature/morganchen12/failable-i…
cbe29ca 
…nitializers

Replace force unwrapping with failable initializer
@hkellaway hkellaway merged commit cbe29ca into develop Aug 22, 2015
@hkellaway hkellaway deleted the feature/morganchen12/failable-initializers branch August 22, 2015 14:46
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
api change enhancement
Projects
None yet
Milestone
0.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@hkellaway @morganchen12