Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SIANXSVC-1214: Use lcov for CodeClimate reports #23

Merged

Conversation

nachtjasmin
Copy link
Collaborator

@nachtjasmin nachtjasmin commented Mar 14, 2024

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.

@nachtjasmin nachtjasmin self-assigned this Mar 14, 2024
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.
@nachtjasmin nachtjasmin force-pushed the chore/SIANXSVC-1214-ensure-working-codeclimate-reports branch from e32f435 to 0f8fcd8 Compare March 14, 2024 08:50
@beachmachine beachmachine merged commit d92bbdf into main Mar 14, 2024
2 checks passed
@beachmachine beachmachine deleted the chore/SIANXSVC-1214-ensure-working-codeclimate-reports branch March 14, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants