Checkstyle severity is ignored while importing, which leads to unintended blockers

[dkreuer]

While importing a checkstyle report the severity is ignored.

Every checkstyle error is mapped to failure, see

CI-Report-Converter/src/Converters/CheckStyleConverter.php

Line 65 in 18ff76a

$case->failure = new SourceCaseOutput($type, $error->get('message'), self::getDetails($error));

This leads to severity "blocker" when generating Gitlab-CodeQuality-Json documents which leads to false information in the Gitlab UI.
A better severity may be "info" in these cases (if not being configurable) - or the default level "major" albeit this is even too much for my liking.

For more context: I create a PHP CodeSniffer Checkstyle Report which has warnings as severity "warning" where warnings do not fail the CI job but errors do.

[SmetDenis]

Hello, @dkreuer

Uh, thanks for the report. I haven't had much experience with Gitlab-CodeQuality-Json. I did the integration based on documentation a long time ago. Maybe their spec has improved and there are new features.

Do you have any sample files for me to play around with?

Perhaps you can do a PR with your enhancement.

Thanks.