Add checkstyle printer

Open sasezaki opened this issue 4 years ago • 3 comments

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

Sep 25 '21 07:09 sasezaki

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

Sep 25 '21 08:09 exussum12

Would prefer this to be an output=xml or output=checkstyle.

Not sure if these should be printed to stderr ?

I will try refactor (includes BC breaks) output format on another PR branch.

Sep 25 '21 13:09 sasezaki

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

I also see this as just a different output

Sep 27 '21 17:09 povils