serde-rs / serde-rs.github.io

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

fix: field-attrs.md #[serde(skip)] and #[serde(skip_serializing)] content invert #164

Closed SoonIter closed 6 months ago

SoonIter commented 6 months ago

Thanks for the PR!

I think this documentation was correct before this PR.

https://github.com/serde-rs/serde-rs.github.io/blob/66d53ef39fcc694456d4ae9621f851acda3cdf31/_src/field-attrs.md?plain=1#L47-L63

I'm sorry, these sentences are not very neat so I think one of these three sentences is not right. Now I know that only deserializing is a special case which needs Default::default().