search

serde-rs / serde-rs.github.io

https://serde.rs
Creative Commons Attribution Share Alike 4.0 International
22 stars 99 forks source link

Document the combined `#[serde(skip)]` field attribute #60

Closed spikefoo closed 7 years ago

spikefoo commented 7 years ago

Document serde#935, and recommend using the combined skip attribute instead of skip_serializing or skip_deserializing.

oli-obk commented 7 years ago

Maybe we should formulate this the opposite way. show #[serde(skip)] as the first attribute of the three, and then state that skip_serializing and skip_deserializing allow more fine-grained control

spikefoo commented 7 years ago

I agree. This commit should make it clear that skip is the main attribute, and that skip_serializing and skip_deserializing are for rare edge cases.

oli-obk commented 7 years ago

Great!