cmu-delphi / delphi-epidata

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

refactor: use delphi_utils.logger instead of copied file #1488

Closed dshemetov closed 5 days ago

dshemetov commented 2 weeks ago

Addresses the comment here. Partial work from #1469.

Summary:

Prerequisites:

sonarcloud[bot] commented 5 days ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

dshemetov commented 5 days ago

I rebased to get recent client changes and separated the whitespace changes in another commit and ignored that commit in blame.

melange396 commented 5 days ago

whoops, the whitespace changes are still showing in blame: https://github.com/cmu-delphi/delphi-epidata/blame/dev/src/acquisition/covidcast/database.py#L125

i didnt realize it before i merged, but i think your force push changed the commit hashes :(