Skip to content

Conversation

@axic
Copy link
Member

@axic axic commented Jan 19, 2019

No description provided.

@axic axic requested a review from jakelang January 19, 2019 03:49
@jakelang
Copy link
Collaborator

What is this used for?

@jakelang jakelang added the ready Ready for review and merge label Jan 21, 2019
Copy link
Collaborator

@jakelang jakelang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but would like to have an example of a use case for this variant.

@axic
Copy link
Member Author

axic commented Jan 21, 2019

LGTM but would like to have an example of a use case for this variant.

One version of #72.

@jakelang
Copy link
Collaborator

ok then it's good, go for merge

@axic axic merged commit c20669a into master Jan 22, 2019
@axic axic deleted the notfound-error branch January 22, 2019 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready Ready for review and merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants