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

Add tests for #130 #145

Merged
merged 1 commit into from Sep 17, 2017
Merged

Add tests for #130 #145

merged 1 commit into from Sep 17, 2017

Conversation

SX91
Copy link
Contributor

@SX91 SX91 commented Sep 15, 2017

No description provided.

adjacently tagged enums serialization/deserialization works with
sequence deserialization.
internally tagged enums deserialization fails with array serialization
(it should fail as serde doesn't support non-mapping formats for
internally tagged enums) but works with map serialization.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 87.076% when pulling 13c42ab on SX91:issue-130 into 8514110 on 3Hren:master.

@3Hren 3Hren merged commit 2a8e04d into 3Hren:master Sep 17, 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.

None yet

3 participants