InfuseAI / piperider

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

[Chore] Remove assertion feature #917

Closed wcchang1115 closed 10 months ago

wcchang1115 commented 10 months ago

PR checklist

What type of PR is this? chore

What this PR does / why we need it:

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

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

codecov[bot] commented 10 months ago

Codecov Report

Merging #917 (ce67823) into main (04e6a2e) will increase coverage by 0.50%. The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main     #917      +/-   ##
==========================================
+ Coverage   47.99%   48.49%   +0.50%     
==========================================
  Files          78       63      -15     
  Lines       10756     9064    -1692     
==========================================
- Hits         5162     4396     -766     
+ Misses       5594     4668     -926     
Files Coverage Δ
piperider_cli/cli.py 68.83% <ø> (+1.23%) :arrow_up:
piperider_cli/cli_utils/run_cmd.py 92.55% <ø> (ø)
piperider_cli/event/events.py 91.30% <ø> (-1.56%) :arrow_down:
piperider_cli/validator.py 23.52% <ø> (+1.30%) :arrow_up:
piperider_cli/runner.py 62.40% <62.50%> (+8.80%) :arrow_up:

... and 2 files with indirect coverage changes

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