Skip to content

CI: fix coverage #895

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Sep 13, 2021
Merged

CI: fix coverage #895

merged 5 commits into from
Sep 13, 2021

Conversation

TriplEight
Copy link
Contributor

@TriplEight TriplEight commented Aug 19, 2021

  • restores code coverage how it used to measure previously - with an old-style instrumentation (but with branch coverage)
  • some chore

CC: #857
Closes: paritytech/ci_cd#160 #863 https://github.com/paritytech/ci_cd/issues/13

@TriplEight TriplEight added B-enhancement New feature or request E-in-progress A task that is already being worked on. A-CI Continuous integration work item labels Aug 19, 2021
@TriplEight TriplEight self-assigned this Aug 19, 2021
@TriplEight
Copy link
Contributor Author

Note: #863 (review)

@TriplEight TriplEight requested review from cmichi and HCastano August 19, 2021 12:33
@codecov-commenter
Copy link

Codecov Report

Merging #895 (de0277b) into master (5571724) will increase coverage by 1.26%.
The diff coverage is 81.80%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #895      +/-   ##
==========================================
+ Coverage   83.33%   84.59%   +1.26%     
==========================================
  Files         163      172       +9     
  Lines        7502     7960     +458     
==========================================
+ Hits         6252     6734     +482     
+ Misses       1250     1226      -24     
Impacted Files Coverage Δ
crates/env/src/api.rs 50.00% <0.00%> (+6.25%) ⬆️
crates/env/src/backend.rs 0.00% <ø> (ø)
...tes/env/src/engine/experimental_off_chain/types.rs 0.00% <0.00%> (ø)
crates/env/src/engine/off_chain/db/accounts.rs 86.90% <ø> (ø)
crates/env/src/engine/off_chain/db/block.rs 100.00% <ø> (ø)
crates/env/src/engine/off_chain/impls.rs 56.33% <0.00%> (+3.70%) ⬆️
crates/env/src/engine/off_chain/test_api.rs 80.85% <0.00%> (ø)
crates/env/src/engine/off_chain/typed_encoded.rs 80.48% <ø> (ø)
crates/env/src/types.rs 44.44% <ø> (ø)
crates/lang/codegen/src/generator/cross_calling.rs 94.82% <ø> (ø)
... and 98 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 eabb117...de0277b. Read the comment docs.

@TriplEight
Copy link
Contributor Author

Feel free to merge if you're OK to do without -Clink-dead-code .

@HCastano
Copy link
Contributor

Feel free to merge if you're OK to do without -Clink-dead-code .

I'm hesitant to merge without this since Robin mentioned that having this flag was correct. Does the Codecov job not pass if this flag is enabled, or what happens?

@TriplEight
Copy link
Contributor Author

Shish, I thought it already works for a month. It could have worked at least somehow. The job breaks with this flag.

@cmichi
Copy link
Collaborator

cmichi commented Sep 13, 2021

I'll merge in the current state and create create a follow-up adding the -Clink-dead-code flag and a fix for the error it causes.

@cmichi cmichi merged commit 1424f0e into master Sep 13, 2021
@cmichi cmichi deleted the 3x8_fix_cov branch September 13, 2021 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI Continuous integration work item B-enhancement New feature or request E-in-progress A task that is already being worked on.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants