Added a Codeclimate output formatter #808
Conversation
marekstodolny
force-pushed
the
codeclimate-formatter
branch
from
4b68678
to
b59195b
Compare
| */ | ||
| private function addUncovered(array &$violationsArray, Uncovered $violation): void | ||
| { | ||
| $violationsArray[] = $this->buildRuleArray($violation, $this->getUncoveredMessage($violation), 'info'); |
There was a problem hiding this comment.
Not sure if you want to report uncovered dependencies as info.
Skipped violations are now set to minor and normal violations are set to major.
Codeclimate spec defines following options: info, minor, major, critical, or blocker and we can choose something different
There was a problem hiding this comment.
If you are unsure about the level, deptrac allows you to define a config for your output formatter. Look at the GrapvizOutputFormatter for reference. You could let the user decide what should be reporting level for any type of dependencies (skipped, normal, uncovered) with some reasonable default.
|
|
|
|
|
||
| #### `severity` | ||
|
|
||
| You can change how a severity of `failure`, `skipped`, `uncovered` violations will be treated. |
There was a problem hiding this comment.
👍 nice touch. I like that
Related to: #807
I based this on the existing JSON output formatter.
Follows this spec: https://github.com/codeclimate/platform/blob/master/spec/analyzers/SPEC.md#data-types