achanda / ipnetwork

A library to work with CIDRs in rust
Apache License 2.0
121 stars 38 forks source link

Make the serde dependency optional #109

Closed faern closed 4 years ago

faern commented 4 years ago

Serde is awesome and supporting serialization for the IpNetwork types is really really handy. But it does not have to be mandatory. With cargo features support it's easy to make it optional. This can cut down compile times.

Personally I would probably not have the feature activated by default. But changing that is a breaking change, so it would have to wait until 0.16 if it's something you think we should do at all.