Flesh out metadata doc schema example

[hyanwong]

As discussed with @benjeffery

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.72%. Comparing base (a0baaaf) to head (051b324).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2146   +/-   ##
=======================================
  Coverage   89.72%   89.72%           
=======================================
  Files          29       29           
  Lines       31567    31573    +6     
  Branches     6113     6115    +2     
=======================================
+ Hits        28322    28328    +6     
  Misses       1853     1853           
  Partials     1392     1392           

Flag	Coverage Δ
c-tests	86.55% <ø> (ø)
lwt-tests	80.78% <ø> (ø)
python-c-tests	88.98% <ø> (ø)
python-tests	99.01% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

[jeromekelleher]

LGTM, but I'd choose a different example in case people think this is the recommended route for location info.

[benjeffery]

Thanks @hyanwong, looks good modulo JK's comments.
One more detail i should comment on - additionalProperties is optional here as it will be added to the schema for you if you use struct. The same thing is happening with required which is being set on the location object in this case.

[hyanwong]

OK, changed to height and age. I left a default for both there, as this is not an obvious thing to be able to do, so having an example is helpful, I think, even though the two values are basically independent.

[hyanwong]

This is now ready for merging, IMO, @benjeffery

[benjeffery]

@Mergifyio rebase

[mergify]

rebase

✅ Nothing to do for rebase action

[benjeffery]

Yeesh 32bit tests are failing hard. Will investigate.

[benjeffery]

@Mergifyio rebase

[mergify]

rebase

✅ Branch has been successfully rebased