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

Visibility #52

Closed
wants to merge 3 commits into from
Closed

Visibility #52

wants to merge 3 commits into from

Conversation

cynecx
Copy link

@cynecx cynecx commented Feb 24, 2020

Fixes #48.

This however requires changes to the MSRV, in particular Rust 1.30 is required. However I am not even sure whether quick-error has a MSRV rule.

Also this PR doesn't include a test because I am not exactly sure how to test this because it's not possible (to my knowledge) to do something like #[should_fail].

@cynecx cynecx requested a review from tailhook February 24, 2020 19:05
@cynecx cynecx closed this Mar 8, 2020
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.

Allow pub(crate)
1 participant