spease / serde_lvm

2 stars 0 forks source link

Crate name #1

Open dtolnay opened 6 years ago

dtolnay commented 6 years ago

The serde_... naming is typically for crates that expose a Serde data format for use with the user's choice of data types. For example serde_json, serde_cbor, serde_yaml, serde_pickle, serde_urlencoded, serde_xml, serde_redis. In all cases the data type being serialized/deserialized is selected by the user by a type parameter like in serde_json::from_reader.

For crates that simply use Serde internally as a detail of the implementation, I would advise against a serde_ prefix.

spease commented 6 years ago

I'll look into getting it renamed to labview_lvm.

Sent from my iPhone

On Apr 29, 2018, at 11:34, David Tolnay notifications@github.com wrote:

The serde_... naming is typically for crates that expose a Serde data format for use with the user's choice of data types. For example serde_json, serde_cbor, serde_yaml, serde_pickle, serde_urlencoded, serde_xml, serde_redis. In all cases the data type being serialized/deserialized is selected by the user by a type parameter like in serde_json::from_reader.

For crates that simply use Serde internally as a detail of the implementation, I would advise against a serde_ prefix.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.