InfuseAI / piperider

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

[Bug] sc-32142 Throw NoneType error when piperider config file is empty #860

Closed kentwelcome closed 1 year ago

kentwelcome commented 1 year ago

PR checklist

What type of PR is this? Bugfix

What this PR does / why we need it: Fix NoneType error when piperider config file is empty

Which issue(s) this PR fixes: sc-32142

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

codecov[bot] commented 1 year ago

Codecov Report

Merging #860 (99d3813) into main (3d44e4b) will not change coverage. The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #860   +/-   ##
=======================================
  Coverage   47.82%   47.82%           
=======================================
  Files          71       71           
  Lines        9994     9994           
=======================================
  Hits         4780     4780           
  Misses       5214     5214           
Files Changed Coverage Δ
piperider_cli/configuration.py 61.36% <0.00%> (ø)

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