compiler-research / CppInterOp

A Clang-based C++ Interoperability Library
Other
37 stars 21 forks source link

Codecov setup #104

Closed Smit1603 closed 1 year ago

Smit1603 commented 1 year ago

-Tried Adding Codecov setup, setup is being established however coverage report is not being generated hence issued a draft PR for your review @vgvassilev and @sudo-panda

vgvassilev commented 1 year ago

I believe you need the relevant cmake changes that build the project with code coverage. They are in clad too.

Smit1603 commented 1 year ago

Yes, had referred it earlier Going through again and finding where the mistake is occuring

Smit1603 commented 1 year ago

Screenshot from 2023-07-09 23-34-52

vgvassilev commented 1 year ago

@Smit1603, can you increase the timeout limit so that you can ssh to the build machine and check if that file is not located somewhere else? In that case we are probably missing the correct path to it.

PS: Why we are processing things from llvm-project/llvm/?

codecov-commenter commented 1 year ago

Codecov Report

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

:exclamation: Current head f5b949f differs from pull request most recent head 001c258. Consider uploading reports for the commit 001c258 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #104   +/-   ##
=======================================
  Coverage        ?   90.16%           
=======================================
  Files           ?       11           
  Lines           ?     2837           
  Branches        ?        0           
=======================================
  Hits            ?     2558           
  Misses          ?      279           
  Partials        ?        0           
Smit1603 commented 1 year ago