Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Ignored tests code in coverage report #615

Merged
merged 1 commit into from
Nov 19, 2021

Conversation

yjhmelody
Copy link
Contributor

Maybe it makes more sense to ignore the coverage of the test part

@codecov
Copy link

codecov bot commented Nov 19, 2021

Codecov Report

Merging #615 (17b5f55) into main (3f12bd6) will decrease coverage by 9.79%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #615      +/-   ##
==========================================
- Coverage   79.55%   69.76%   -9.80%     
==========================================
  Files         408      297     -111     
  Lines       24856    16569    -8287     
==========================================
- Hits        19774    11559    -8215     
+ Misses       5082     5010      -72     
Impacted Files Coverage Δ
src/compute/sort/primitive/indices.rs 75.00% <0.00%> (-19.83%) ⬇️
src/compute/comparison/utf8.rs 67.56% <0.00%> (-14.53%) ⬇️
src/compute/comparison/binary.rs 67.56% <0.00%> (-14.53%) ⬇️
src/io/parquet/read/schema/convert.rs 77.96% <0.00%> (-11.69%) ⬇️
src/types/bit_chunk.rs 40.90% <0.00%> (-7.10%) ⬇️
src/compute/take/utf8.rs 90.90% <0.00%> (-6.15%) ⬇️
src/compute/comparison/boolean.rs 87.23% <0.00%> (-5.36%) ⬇️
src/io/parquet/read/schema/metadata.rs 76.66% <0.00%> (-5.16%) ⬇️
src/compute/take/list.rs 95.00% <0.00%> (-4.07%) ⬇️
src/compute/take/boolean.rs 90.38% <0.00%> (-2.58%) ⬇️
... and 123 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f12bd6...17b5f55. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 2363b27 into jorgecarleitao:main Nov 19, 2021
@jorgecarleitao jorgecarleitao added the testing PRs that only increase coverage label Nov 19, 2021
@jorgecarleitao
Copy link
Owner

Thanks!

@jorgecarleitao jorgecarleitao changed the title chore(ci): add ignore-tests for coverage Ignored tests code in coverage report Nov 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
testing PRs that only increase coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants