InfuseAI / piperider

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

[Feature] sc-32095 Add `--skip-datasource` option for run command #855

Closed kentwelcome closed 1 year ago

kentwelcome commented 1 year ago

PR checklist

What type of PR is this? Feature

What this PR does / why we need it:

Loom Record https://www.loom.com/share/225de0c09b2842fc84fd2ec5919a3ee4?sid=57033864-0559-4521-a96e-fba9c27b1539

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

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

codecov[bot] commented 1 year ago

Codecov Report

Merging #855 (bb88a5c) into main (388621d) will increase coverage by 2.77%. Report is 1 commits behind head on main. The diff coverage is 68.03%.

@@            Coverage Diff             @@
##             main     #855      +/-   ##
==========================================
+ Coverage   45.10%   47.87%   +2.77%     
==========================================
  Files          70       71       +1     
  Lines        9851     9944      +93     
==========================================
+ Hits         4443     4761     +318     
+ Misses       5408     5183     -225     
Files Changed Coverage Δ
piperider_cli/profiler/event.py 74.24% <50.00%> (-5.39%) :arrow_down:
piperider_cli/runner.py 55.04% <55.35%> (+34.64%) :arrow_up:
piperider_cli/configuration.py 61.36% <66.66%> (+0.48%) :arrow_up:
piperider_cli/datasource/unsupported.py 80.95% <80.95%> (ø)
piperider_cli/cli.py 64.01% <100.00%> (+0.09%) :arrow_up:
piperider_cli/cloud/__init__.py 25.72% <100.00%> (ø)
piperider_cli/datasource/__init__.py 55.38% <100.00%> (+0.12%) :arrow_up:
piperider_cli/error.py 56.37% <100.00%> (-0.18%) :arrow_down:
piperider_cli/profiler/profiler.py 74.61% <100.00%> (+2.47%) :arrow_up:

... and 5 files with indirect coverage changes

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