Skip to content

Commit

Permalink
SIANXSVC-1214: Use lcov for CodeClimate reports
Browse files Browse the repository at this point in the history
Even though the Code Coverage report is correct and other platforms like
GitLab do not have a problem understanding it, the CodeClimate reporter
struggles with double slashes that seem to be introduced by itself.

This is also tracked in an issue, which is open since three years with
no intent of fixing it. Therefore, I assume this is not going to be
closed soon.

codeclimate/test-reporter#458

As a workaround, we're now using the lcov format.
  • Loading branch information
nachtjasmin committed Mar 14, 2024
1 parent abaf195 commit 0f8fcd8
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,7 @@ jobs:
run: >
dotnet test
--no-restore
--collect:"XPlat Code Coverage"
/p:CoverletOutputFormat=cobertura
--collect:"XPlat Code Coverage;Format=lcov"
--logger trx
--results-directory "test-results"
--
Expand All @@ -39,4 +38,4 @@ jobs:
env:
CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
with:
coverageLocations: "test-results/**/coverage.cobertura.xml:cobertura"
coverageLocations: "test-results/**/coverage.info:lcov"

0 comments on commit 0f8fcd8

Please sign in to comment.