Change severity in Checkstyle output to lowercase #624
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi! 👋
I'm currently adding this GitHub Action to our pipeline to display and annotate the code with the results from the scapegoat-run. Unfortunately, everything is just an "Annotation", because the GH Action expects the
severity
in the xml to be lowercase.The DTD unfortunately doesn't define the
severity
attribute (https://checkstyle.org/dtds/configuration_1_3.dtd), butgoogle_checks
usewarning
as default: https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/google_checks.xml#L22sun_checks
useerror
as default: https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/sun_checks.xml#L42So I think this change would make the output "more compliant" and make parsing the xml a but easier 😊