screaming kebab case doesn't treat numbers as separate word?

[Luro02]

The following code doesn't work as expected

#[derive(Display)]
#[strum(serialize_all = "SCREAMING-KEBAB-CASE")]
enum Example {
    Aes128
}

assert_eq!(Example::Aes128, "AES-128".to_string());

instead strum generates:
"AES128"

[Peternator7]

Thanks for reaching out!

Strum delegates generating all of it's different cases to the heck crate. While I agree that's unexpected, I'm hesitant to add special cases on top of that library. I might suggest reaching out on that repo and see if they'd take a fix in their crate.

Thanks!

[jplatte]

Might I suggest moving away from heck? It uses unicode word boundaries, which seems excessive for source code (I haven't seen anybody complain about serde's rename_all lacking support for #[feature(non_ascii_idents)] and I doubt anybody currently using strum needs that).

EDIT: I'm now maintaining the heck crate and will soon release a new version with the unicode-segmentation dependency being optional (but am still unsure whether it makes sense for strum to depend on it).

Maybe you could reuse serde's implementation or coordinate a shared crate for this logic with them?

That implementation does however also not put -s or _s between letters and digits. I would strongly suggest not deviating from serde's rename_all, no matter if you're using heck or not.