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 serde::Serialize and serde::Deserialize for Ksuid structs. #4

Merged
merged 1 commit into from
Oct 11, 2022

Conversation

mrl5
Copy link
Contributor

@mrl5 mrl5 commented Oct 8, 2022

Motivation

  1. as a rust-ksuid lib user I want to serialize to JSON structs that use Ksuid type
  2. as a rust-ksuid lib user I want to deserialize from JSON to structs that use Ksuid type

Solution

Implement serde::Serialize and serde::Deserialize for Ksuid structs.

@mrl5 mrl5 force-pushed the implement-serde-traits branch 2 times, most recently from 5ba172d to 7496d9d Compare October 8, 2022 23:23
Copy link
Member

@tasn tasn left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, looks great. I just made a couple of small comments.

Makefile Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@mrl5 mrl5 requested a review from tasn October 10, 2022 17:45
this change covers this usecases:
1. as a `rust-ksuid` lib user I want to serialize to JSON structs that
   use Ksuid type
2. as a `rust-ksuid` lib user I want to deserialize from JSON to structs
   that use Ksuid type
@tasn
Copy link
Member

tasn commented Oct 11, 2022

Ready to be merged?

@tasn tasn merged commit 1646800 into svix:main Oct 11, 2022
@mrl5 mrl5 deleted the implement-serde-traits branch October 11, 2022 20:27
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