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

Parameterize by the error type #15

Merged
merged 3 commits into from Feb 25, 2015
Merged

Parameterize by the error type #15

merged 3 commits into from Feb 25, 2015

Conversation

robrix
Copy link
Contributor

@robrix robrix commented Feb 25, 2015

Result<T> is now Result<T, Error>.

robrix added a commit that referenced this pull request Feb 25, 2015
Parameterize by the error type
@robrix robrix merged commit 7710ed7 into master Feb 25, 2015
@robrix robrix deleted the wrap-either branch February 25, 2015 15:54
@robrix
Copy link
Contributor Author

robrix commented Mar 31, 2015

@rnapier Any thoughts on the type now that this change is in place? Do you think this is ready to be a successor to LlamaKit?

@rnapier
Copy link
Member

rnapier commented Mar 31, 2015

This looks good. I think we're ready to pull the trigger.

On Tue, Mar 31, 2015 at 10:37 AM, Rob Rix notifications@github.com wrote:

@rnapier https://github.com/rnapier Any thoughts on the type now that
this change is in place? Do you think this is ready to be a successor to
LlamaKit?


Reply to this email directly or view it on GitHub
#15 (comment).

Rob Napier
Cocoaphony blog -- http://robnapier.net/blog
iOS Programming Pushing the Limits -- http://robnapier.net/book

@robrix
Copy link
Contributor Author

robrix commented Mar 31, 2015

👍

@rnapier
Copy link
Member

rnapier commented Mar 31, 2015 via email

@robrix
Copy link
Contributor Author

robrix commented Mar 31, 2015

Just point people right at this repo. I’ll add it to antitypical.github.io when I get to that 😊

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

2 participants