Add checkstyle printer #139
Conversation
| The ``--checkstyle-output`` option will generate a report in checkstyle xml format to the path specified by the option. | ||
|
|
||
| The ``--extensions`` option lets you extend the code analysis. The provided extensions must be comma separated. | ||
|
|
||
| **By default it analyses conditions, return statements, and switch cases.** |
There was a problem hiding this comment.
This line "**By default it analyses ~", was seemed to intended to add after --extensions . So I move "The --extensions ~" line.
|
Would prefer this to be an output=xml or output=checkstyle. You could pass both flags currently which would be strange- Checkstyle is a great idea though |
|
|
||
| public function printData(OutputInterface $output, FileReportList $fileReportList, HintList $hintList): void | ||
| { | ||
| $output->writeln('Generate checkstyle report output...'); |
There was a problem hiding this comment.
Not sure if these should be printed to stderr ?
There was a problem hiding this comment.
I just did find what you pointed :)
squizlabs/PHP_CodeSniffer#1612
I will try refactor (includes BC breaks) output format on another PR branch. |
I also see this as just a different output |
This PR requests adding checkstyle output feature.
I need checkstyle output to work with cs2pr, and other QA tools.
Notes: This PR also inclueds cs-fix