Closed Dimi1010 closed 1 month ago
Attention: Patch coverage is 92.85714%
with 24 lines
in your changes missing coverage. Please review.
Project coverage is 82.88%. Comparing base (
0aa0c83
) to head (3472012
). Report is 3 commits behind head on dev.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
LGTM but didn't check all the files, @Dimi1010 did you use CPPcheck to do this ?
LGTM but didn't check all the files, @Dimi1010 did you use CPPcheck to do this ?
Yes, when I remembered about it. Started by searching for them manually initially.
override
specifier.= default
instead of{}
missingOverrides
cppcheck suppression from everywhere exceptExamples
andPcap++
as those are the only modules that currently raiseoverride
warnings.