Make reporter option require outputFile option

[Tapchicoma]

I've tried to introduce new reporter option in the https://github.com/JLLeitschuh/ktlint-gradle plugin and one problem so far - now console output lacks old plain errors when reporter is set to checkstyle or json.

I'd like to have old behaviour for ktlint run output, but introduce new option --outputFile to where reporter will write it output.

[shyiko]

@Tapchicoma sounds reasonable.

Instead of introducing another CLI option, how about something like
ktlint --reporter=plain?group_by_file --reporter="checkstyle > target/checkstyle.xml" --reporter="html > target/ktlint.html"?

[Tapchicoma]

Hm. If you don't want to introduce another option then I'd like to have more standard approach for additional command line option arguments:

ktlint --reporter plain?group_by_file --reporter checkstyle=target/checkstyle.xml

or

ktlint --reporter plain?group_by_file --reporter checkstyle,output=target/checkstyle.xml

See getsubopt() documentation for reference.

[shyiko]

@Tapchicoma it's not that I don't want to introduce another option, it just --outputFile can potentially make things complicated for multi-format reporting (e.g. checkstyle and html at the same time).
Anyway, I like your 2nd example (ktlint --reporter plain?group_by_file --reporter checkstyle,output=target/checkstyle.xml). I think this is the way to go.
Also, thank you for the link 🙇‍♂️.

[shyiko]

Implemented in 0.10.0.