InfuseAI / piperider

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

[Bug] Fix threads parsing bugs #909

Closed qrtt1 closed 10 months ago

qrtt1 commented 10 months ago

Fixes #907

PR checklist

What type of PR is this?

bugfix

What this PR does / why we need it:

Make sure the threads is an integer value.

Which issue(s) this PR fixes:

sc-32473

codecov[bot] commented 10 months ago

Codecov Report

Merging #909 (4dd1591) into main (62890e7) will decrease coverage by 0.03%. The diff coverage is 29.41%.

@@            Coverage Diff             @@
##             main     #909      +/-   ##
==========================================
- Coverage   47.97%   47.95%   -0.03%     
==========================================
  Files          77       77              
  Lines       10726    10739      +13     
==========================================
+ Hits         5146     5150       +4     
- Misses       5580     5589       +9     
Files Coverage Δ
piperider_cli/__init__.py 51.86% <50.00%> (-0.31%) :arrow_down:
piperider_cli/datasource/__init__.py 53.73% <0.00%> (-1.66%) :arrow_down:

... and 1 file with indirect coverage changes

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