ci: validate that schemas are up to date

[joanise]

No description provided.

[github-actions]

CLI load time: 0:00.30
Pull Request HEAD: f2db84faf87a7420550fec7523513480f7ad5f55
Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.64%. Comparing base (d65e00f) to head (f2db84f).
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #389      +/-   ##
==========================================
- Coverage   73.19%   72.64%   -0.56%     
==========================================
  Files          43       43              
  Lines        2791     2829      +38     
  Branches      459      474      +15     
==========================================
+ Hits         2043     2055      +12     
- Misses        664      689      +25     
- Partials       84       85       +1     

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

[roedoejet]

great! nice and easy, this is all we need. Although maybe we should have a message that says that if the schemas change, we should bump the minor version number too?

[joanise]

Hum, everyvoice update-schemas will already tell you that if you run it and the schemas exist. It seems redundant to say it again here.

[roedoejet]

This looks good to me, but should we not indicate somewhere that when the schemas change, the minor version number ought to change too?

[joanise]

This looks good to me, but should we not indicate somewhere that when the schemas change, the minor version number ought to change too?

Yes, the warning everyvoice update-schemas prints when the schemas already exist is that somewhere. Is that inadequate?

[roedoejet]

looks good!