Closed vgvassilev closed 10 months ago
Merging #142 (d861985) into main (562533a) will increase coverage by
0.04%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #142 +/- ##
==========================================
+ Coverage 64.55% 64.59% +0.04%
==========================================
Files 16 16
Lines 4212 4214 +2
==========================================
+ Hits 2719 2722 +3
+ Misses 1493 1492 -1
Files Changed | Coverage Δ | |
---|---|---|
include/clang/Interpreter/Compatibility.h | 90.19% <100.00%> (+2.19%) |
:arrow_up: |
lib/Interpreter/CppInterOp.cpp | 75.26% <100.00%> (+0.01%) |
:arrow_up: |
Files Changed | Coverage Δ | |
---|---|---|
include/clang/Interpreter/Compatibility.h | 90.19% <100.00%> (+2.19%) |
:arrow_up: |
lib/Interpreter/CppInterOp.cpp | 75.26% <100.00%> (+0.01%) |
:arrow_up: |
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
The build fails but the upstream fix appeared recently so maybe it is a hiccup. Let's merge this PR.
Currently it is rc3 and will be released next week.