Combine serde field attributes with smarter OutputMember::field_annot

[kube-rs]

Combine serde field attributes with smarter OutputMember::field_annot

[clux]

Although, if rustfmt can combine field attributes like this, then we don't have to do this. Not sure if this is true.

EDIT: not seeing anything at the moment. merge_derives is the most similar one, but we already do that. I guess this would be hard for rustfmt to do since the syntax inside attributes is partially up to the user?

[clux]

Well I tried this in a branch and I can indeed produce:

#[derive(Serialize, Deserialize, Clone, Debug)]
pub struct ServerPodSelector {
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "matchExpressions")]
    pub match_expressions: Option<Vec<ServerPodSelectorMatchExpressions>>,
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "matchLabels")]
    pub match_labels: Option<BTreeMap<String, serde_json::Value>>,
}

but unfortunately, it's way more common for these to be force split by rustfmt down the line when slightly longer:

here servicemonitors:

pub struct ServiceMonitorSpec {
    pub endpoints: Vec<ServiceMonitorEndpoints>,
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "jobLabel")]
    pub job_label: Option<String>,
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "labelLimit")]
    pub label_limit: Option<i64>,
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "labelNameLengthLimit")]
    pub label_name_length_limit: Option<i64>,
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "labelValueLengthLimit")]
    pub label_value_length_limit: Option<i64>,
    /// ...
}

becomes

pub struct ServiceMonitorSpec {
    pub endpoints: Vec<ServiceMonitorEndpoints>,
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "jobLabel")]
    pub job_label: Option<String>,
    #[serde(default, skip_serializing_if = "Option::is_none", rename = "labelLimit")]
    pub label_limit: Option<i64>,
    #[serde(
        default,
        skip_serializing_if = "Option::is_none",
        rename = "labelNameLengthLimit"
    )]
    pub label_name_length_limit: Option<i64>,
    #[serde(
        default,
        skip_serializing_if = "Option::is_none",
        rename = "labelValueLengthLimit"
    )]
    pub label_value_length_limit: Option<i64>,
    /// ...
}

[clux]

Not sure which is best, POVs welcome.

[clux]

Going to smash them together because it's nicer to work with api wise. If people find the output ugly they can change line limit params in rustfmt.