Skip to content
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

CI/CD: Gather code test coverage. #1124

Merged
merged 3 commits into from
Nov 26, 2020
Merged

CI/CD: Gather code test coverage. #1124

merged 3 commits into from
Nov 26, 2020

Conversation

briansmith
Copy link
Owner

@briansmith briansmith commented Nov 25, 2020

Gather code review data to help guide the code review process. This is currently limited to i686, x86_64, and AArch64 Linux because these are the targets I could get working. More investigation is needed to figure out how to collect coverage data for ARM targets.

@briansmith briansmith self-assigned this Nov 25, 2020
@briansmith briansmith changed the title CI/CD: Gather code coverage. CI/CD: Gather code test coverage. Nov 25, 2020
Gather code review data to help guide the code review process. This is currently limited
to i686 and x86_64 Linux because these are the targets I could get working. More
investigation is needed to figure out how to collect coverage data for ARM targets.
@codecov-io
Copy link

codecov-io commented Nov 25, 2020

Codecov Report

❗ No coverage uploaded for pull request base (main@db664fa). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1124   +/-   ##
=======================================
  Coverage        ?   82.40%           
=======================================
  Files           ?       90           
  Lines           ?    10390           
  Branches        ?        0           
=======================================
  Hits            ?     8562           
  Misses          ?     1828           
  Partials        ?        0           

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 db664fa...b57649f. Read the comment docs.

aarch64-unknown-linux-musl doesn't have profiler_builtins enabled so
use aarch64-unknown-linux-gnu instead.

Work around an issue where the "%m" feature of the LLVM source-based code
coverage doesn't work within QEMU.
@briansmith briansmith force-pushed the b/cov-llvm-source branch 2 times, most recently from e77c8f2 to b57649f Compare November 25, 2020 21:38
@briansmith briansmith mentioned this pull request Nov 25, 2020
5 tasks
@briansmith
Copy link
Owner Author

briansmith commented Nov 25, 2020

Issue #132 tracks improvements to the code test coverage measurement.

@briansmith briansmith merged commit 08c5854 into main Nov 26, 2020
@briansmith briansmith deleted the b/cov-llvm-source branch November 26, 2020 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants