search

tomasbjerre / violations-lib

Java library for parsing report files from static code analysis.
Apache License 2.0
148 stars 39 forks source link

`CodeClimateParser` should map `check_name` to `rule` #132

Closed remal closed 3 years ago

remal commented 3 years ago

CodeClimate class has check_name field that is supposed to handle rule name: https://github.com/tomasbjerre/violations-lib/blob/f24f5293544ae9f9886de267140819d3e9cb0884/src/main/java/se/bjurr/violations/lib/model/codeclimate/CodeClimate.java#L18

However, CodeClimateParser doesn't handle check_name field: https://github.com/tomasbjerre/violations-lib/blob/f24f5293544ae9f9886de267140819d3e9cb0884/src/main/java/se/bjurr/violations/lib/parsers/CodeClimateParser.java#L48-L56

Could you please add check_name to rule mapping?

tomasbjerre commented 3 years ago

fixed, open issue again if any problems.