InfuseAI / piperider

Code review for data in dbt
https://www.piperider.io/
Apache License 2.0
480 stars 23 forks source link

[Bug] Fix upgrade_schema bug to keep metrics data #871

Closed qrtt1 closed 1 year ago

qrtt1 commented 1 year ago

PR checklist

What type of PR is this?

bugfix

What this PR does / why we need it:

we need the metrics

Which issue(s) this PR fixes:

sc-32169

codecov[bot] commented 1 year ago

Codecov Report

Merging #871 (af1db93) into main (2ed2b8f) will increase coverage by 0.06%. The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main     #871      +/-   ##
==========================================
+ Coverage   46.32%   46.39%   +0.06%     
==========================================
  Files          75       75              
  Lines       10292    10306      +14     
==========================================
+ Hits         4768     4781      +13     
- Misses       5524     5525       +1     
Files Changed Coverage Δ
piperider_cli/dbt/list_task.py 65.45% <86.66%> (+0.82%) :arrow_up:
piperider_cli/runner.py 54.26% <100.00%> (ø)

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more