Codecov Report

Attention: Patch coverage is 71.42857% with 10 lines in your changes missing coverage. Please review.

Project coverage is 76.27%. Comparing base (64b8133) to head (0bcbd44).
Report is 5 commits behind head on main.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   76.27%   76.27%   -0.01%     
==========================================
  Files          15       15              
  Lines        4274     4282       +8     
  Branches     4274     4282       +8     
==========================================
+ Hits         3260     3266       +6     
+ Misses        872      871       -1     
- Partials      142      145       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Hi @twop ! I should have the chance to look at this in more detail over the weekend, but on a design note (and sorry if this might add a bit of additional work), what do you think about also adding, for completeness, something like serdes rename_all attribute?

https://serde.rs/container-attrs.html#rename_all

That might be an even more elegant way to address your particular case where you want everything in PascalCase!

I did think about it, but decided not to tackle it in this PR:

1. I personally prefer explicit field renames with serde(although, that is a preference).
2. I think if people don't ask for it => don't build it. Even if it makes sense, I would probably still prefer to introduce it in a separate PR.

That said, it is my preference, and not a belief.

Upd: added explicit naming for child too

I did think about it, but decided not to tackle it in this PR:

As for this, my personal preference likely runs counter to this — I think I'd be frustrated by having that "boilerplate" for every field, but, with that being said, it's not a feature I personally need right now.

Ideally we'd also have that "rename all" support (also means more intuitive coming from serde), but I don't mind merging this on its own either!

rename all, seems nice, but I haven't looked into it's implementation