Closed vgvassilev closed 10 months ago
clang-tidy review says "All clean, LGTM! :+1:"
Merging #148 (f6477bd) into main (bcd51b7) will increase coverage by
0.14%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## main #148 +/- ##
==========================================
+ Coverage 64.49% 64.64% +0.14%
==========================================
Files 16 17 +1
Lines 4247 4279 +32
==========================================
+ Hits 2739 2766 +27
- Misses 1508 1513 +5
Files Changed | Coverage Δ | |
---|---|---|
unittests/CppInterOp/CUDATest.cpp | 50.00% <ø> (ø) |
... and 2 files with indirect coverage changes
Files Changed | Coverage Δ | |
---|---|---|
unittests/CppInterOp/CUDATest.cpp | 50.00% <ø> (ø) |
The llvm17 build still has valgrind issues in the llvm release branch itself. Let's merge this even though the bot fails.
This reverts commit bcd51b7b00a9df4121f86673b552241d6a0c93b4 which was accidentally merged.