Fixed CI/CD coverage #393
Fixed CI/CD coverage #393
Conversation
made the action run on fix/CI_CD typo fix added lcov dependency made lcov a separate action added gcovr as alternative to the lcov that doesn't work well with macro coverage errors ignoring made full switch to gcovr removed lcov and glaze that was from other pr fix of wrong configuring of preset possible fix fix made gcovr generate .info file fix made cmake use current binary dir propper syntax for output file tried fixing build folder inclusion naming fix switched to the new coverage
|
This use to work with no code changes. I suspect the problem is the gcc or llvm and lcov in the github runner are incompatible. In my testing they are different versions. The output has been super consistent but its been a while since I've looked at this prince-chrismc@8c83693 Why did the cmake module being used not work, is has support for both tools? |
|
Yes the lint needs to be applied manually, there's a cli in the error message. Automatically with githubs permission model is not sensible |
Hmm, If i knew the commit at which the CI started failing, I'll go look into it, maybe the answer is in there, a lot of things started failing when we at our project updated gcc maybe here is something alike.
I'll try it out, couldn't find the info on the gcovr, might have to research that bit, was pretty late when i was finishing things up. |
I managed to fix the CI/CD pipeline for coverage by switching to gcovr, so if this is not something you want to do, tell me i'll try to find a solution to lcov, but i've tried updating versions and many other things and it still fails because of line endings, like in this issue but then it requires more and more ignores, at some point it doesn't look like coverage, here is the resulting link coveralls,
the Lint is still failing, but it looks like its by design. so the changes would be applied only manually?