InfuseAI / piperider

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

[Feature] Support skip-datasource option for piperider compare #885

Closed wcchang1115 closed 1 year ago

wcchang1115 commented 1 year ago

PR checklist

What type of PR is this? feature

What this PR does / why we need it:

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

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

codecov[bot] commented 1 year ago

Codecov Report

Merging #885 (90db541) into main (0a4839a) will increase coverage by 1.74%. Report is 4 commits behind head on main. The diff coverage is 55.55%.

@@            Coverage Diff             @@
##             main     #885      +/-   ##
==========================================
+ Coverage   46.41%   48.16%   +1.74%     
==========================================
  Files          75       75              
  Lines       10358    10390      +32     
==========================================
+ Hits         4808     5004     +196     
+ Misses       5550     5386     -164     
Files Changed Coverage Δ
piperider_cli/cli_utils/compare_with_recipe.py 0.00% <0.00%> (ø)
piperider_cli/recipe_executor.py 0.00% <0.00%> (ø)
piperider_cli/recipes/github_action.py 0.00% <0.00%> (ø)
piperider_cli/recipes/__init__.py 26.27% <53.84%> (+26.27%) :arrow_up:
piperider_cli/recipes/default_recipe_generator.py 34.52% <75.00%> (+34.52%) :arrow_up:
piperider_cli/cli.py 65.21% <100.00%> (+0.10%) :arrow_up:
piperider_cli/runner.py 54.26% <100.00%> (ø)

... and 4 files with indirect coverage changes

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