InfuseAI / piperider

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

[Bug] Make the ruamel integeration compatiable with legacy versions #925

Closed qrtt1 closed 9 months ago

qrtt1 commented 9 months ago

Fix #923

PR checklist

What type of PR is this?

bugfix

What this PR does / why we need it:

Which issue(s) this PR fixes:

sc-32539

Special notes for your reviewer:

We also re-arrange the test matrix by tox.

codecov[bot] commented 9 months ago

Codecov Report

Merging #925 (3b558cf) into main (02b474b) will decrease coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #925      +/-   ##
==========================================
- Coverage   48.48%   48.48%   -0.01%     
==========================================
  Files          63       63              
  Lines        9066     9064       -2     
==========================================
- Hits         4396     4395       -1     
+ Misses       4670     4669       -1     
Files Coverage Δ
piperider_cli/yaml/__init__.py 72.97% <100.00%> (-1.39%) :arrow_down:

... and 1 file with indirect coverage changes

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

popcornylu commented 9 months ago

Fix #923