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

Implement JsonSchema on EnumSet type #92

Closed
wants to merge 2 commits into from

Conversation

mwcampbell
Copy link
Contributor

The enumset combines the efficiency of a bitmask with the ergonomics of a set of enum values. I'd like to use it with schemars, so I'm contributing support.

@mwcampbell
Copy link
Contributor Author

Is there anything else I need to do before this can be merged, or anything I can do to make it easier to review and merge?

@GREsau
Copy link
Owner

GREsau commented Oct 10, 2021

Thanks @mwcampbell, I've merged this in d059686

I didn't merge the second commit since the tests appear to work just fine on rust 1.37, I don't suppose you remember why you originally needed to exclude the tests from 1.37?

@GREsau GREsau closed this Oct 10, 2021
@mwcampbell
Copy link
Contributor Author

I think I must have been over-reacting to an intermittent CI failure.

@GREsau Is there anything blocking you from publishing a new release on crates.io now?

@GREsau
Copy link
Owner

GREsau commented Nov 14, 2021

Sorry for the delay, this is now on crates.io in 0.8.7!
https://crates.io/crates/schemars/0.8.7

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