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

Make error module public #6

Merged
merged 2 commits into from
Sep 7, 2017
Merged

Conversation

dgrnbrg
Copy link
Contributor

@dgrnbrg dgrnbrg commented Sep 4, 2017

I am integrating this library into another library, and I'd like to participate in the full power of error chain. However, I can't link against your errors since the module's private. This change makes it possible for other projects to get rich error information programmatically.

dgrnbrg and others added 2 commits September 4, 2017 11:42
I am integrating this library into another library, and I'd like to participate in the full power of error chain. However, I can't link against your errors since the module's private. This change makes it possible for other projects to get rich error information programmatically.
@dgrnbrg
Copy link
Contributor Author

dgrnbrg commented Sep 4, 2017

I also updated the version of error-chain to support the newer syntax. One issue I ran into is that the strict documentation requirement conflicts with newer error chain, so I removed that pragma.

@AtheMathmo
Copy link
Owner

Thanks for this - looks good to me!

@AtheMathmo AtheMathmo merged commit e4e3c04 into AtheMathmo:master Sep 7, 2017
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.

2 participants