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.

Sign up for GitHub

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

Jump to bottom

Add support for Serde #37

Closed
iliekturtles opened this issue Dec 20, 2017 · 1 comment
Closed

Add support for Serde #37

iliekturtles opened this issue Dec 20, 2017 · 1 comment

Comments

@iliekturtles
Copy link
Owner

Add support controlled by a serde feature. Enable or disable by default?

https://crates.io/crates/serde
@radix
Copy link
Contributor

radix commented Dec 23, 2017

As a user of serde in an application, I think it's reasonable to only enable it when a feature is explicitly enabled. I'm used to doing this for other crates.

@radix radix mentioned this issue Dec 28, 2017
Closed
radix added a commit to radix/uom that referenced this issue Dec 28, 2017
@radix
Implement serde::ser::Serialize and serde::de::Deserialize.
9b040e0 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 28, 2017
@radix
Implement serde::ser::Serialize and serde::de::Deserialize.
8e936c4 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 28, 2017
@radix
Implement serde::ser::Serialize and serde::de::Deserialize.
cd81039 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 28, 2017
@radix
Implement serde::ser::Serialize and serde::de::Deserialize.
0ad4ace 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 28, 2017
@radix
Implement serde::Serialize and serde::Deserialize.
ee6f509 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 28, 2017
@radix
Implement serde::Serialize and serde::Deserialize.
faca99e 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 28, 2017
@radix
Implement serde::Serialize and serde::Deserialize.
a727657 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 29, 2017
@radix
Implement serde::Serialize and serde::Deserialize.
c0d8f78 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Dec 29, 2017
@radix
Implement serde::Serialize and serde::Deserialize.
0b75490 
Fixes iliekturtles#37.
radix added a commit to radix/uom that referenced this issue Jan 1, 2018
@radix
Implement serde::Serialize and serde::Deserialize.
8ff1d87 
Fixes iliekturtles#37.
iliekturtles pushed a commit that referenced this issue Jan 2, 2018
@radix @iliekturtles
Implement serde::Serialize and serde::Deserialize.
fd14b3b 
Fixes #37.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@radix @iliekturtles