search

cellml / libcellml

Repository for libCellML development.
https://libcellml.org
Apache License 2.0
16 stars 21 forks source link

CMake: `coverage_branches` target needed? #955

Closed agarny closed 3 years ago

agarny commented 3 years ago

Is the coverage_branches target needed? It looks like we don't use it in our CI tests?

hsorby commented 3 years ago

It isn't needed as such but branch information is useful in understanding code coverage.

agarny commented 3 years ago

So, we don't make use of it in our CI tests, but we keep that CMake target in case people want to get a bit more information? I guess there is nothing wrong with that per se, so closing.