-
Notifications
You must be signed in to change notification settings - Fork 86
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
Removing Argo #91
Removing Argo #91
Conversation
Has anyone had a chance to take a look at this? 😁 |
@@ -155,10 +155,9 @@ public let HeimdallrErrorNotAuthorized = 2 | |||
completion(.Failure(error)) | |||
} | |||
} else { | |||
switch OAuthError.decode(data!) { | |||
case let .Success(error): | |||
if let error = OAuthError.decode(data: data!) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you mind manually unwrapping data
, potentially failing by not satisfying a condition or using fatalError
with an appropriate error message if data
is nil
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felixjendrusch Done in 4ab6083
I would give it a go. I do not think that the dependency update should do any harm. @felixjendrusch Green from your side? And once more: Thank you @marcelofabri and a big 👍 for contributing. |
@hffmnn yo. Thanks @marcelofabri. |
This fixes #90
I wasn't able to figure out a way to make Carthage just remove Argo, so I had to run
carthage update ...
, which updated Nimble, Result and ReactiveCocoa. I hope it's not a problem.