libcellml icon indicating copy to clipboard operation
libcellml copied to clipboard

Published 20 hours ago • verified publisher icon cellml

CMake: LLVM coverage should require unit tests

Open agarny opened this issue 3 years ago • 1 comments

As for coverage and memcheck, LLVM coverage should require unit tests to be enabled.

agarny avatar Aug 03 '21 05:08 agarny

I would have though that we should not make coverage and memcheck require unit tests. These settings don't require unit tests it is just convenient that they do from a CI perspective.

Either way, we should at least be consistent. We don't want to show bias against LLVM.

hsorby avatar Aug 27 '21 23:08 hsorby