cmu-delphi / delphi-epidata

An open API for epidemiological data.
https://cmu-delphi.github.io/delphi-epidata/
MIT License
100 stars 68 forks source link

One-time version check #1456

Closed rzats closed 2 months ago

rzats commented 4 months ago

Closes #1281

Summary:

Adds a one-time check to the Epidata client, which logs a warning when the current "remote" server version (https://api.delphi.cmu.edu/epidata/version) does not match the client version.

This is limited to debug mode for now but can be easily extended to run by default.

Also adds a test to verify this behavior.

Prerequisites:

melange396 commented 4 months ago

Let's do #1296 first and then you can revisit this

rzats commented 3 months ago

@melange396 since #1296 is merged, we can move forward with this PR; lmk if the current iteration looks good!

sonarcloud[bot] commented 2 months ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud