-
Notifications
You must be signed in to change notification settings - Fork 1
Description
The mapping for lint severities is: Error->Blocker, Warning->Critical, Else->Info https://github.com/MobileUpLLC/CodeQuality-Android/blob/master/src/main/kotlin/ru/mobileup/codequality/input/LintParser.kt#L31
At the first glance this feels odd to me since I translate Error as something I must look into and Warning as something I should look into. Critical sounds like more than something i should look into.
When looking into the documentation which Severities exist:
Lint has info, warning, error and fatal https://developer.android.com/studio/write/lint.
Codeclimate has info, minor, major, critical and blocker https://docs.codeclimate.com/docs/issues#issue-severity.
We can also look how sonarqube is mapping lint to its severities (minor, major, critical), that are similar to codeclimate.
e.g. 
Warning -> Minor, Error -> Major
I suggest to adjust the severity mapping to:
info->info, warning->minor, error->major, fatal->critical