Roman Ivanov
Roman Ivanov
@rnveach , are you aware about this PR ? it is related to your PR https://github.com/checkstyle/contribution/pull/747
yes we need to update script to work with modern Groovy.
@rnveach , please create PR to main repo to use your branch and validate and action is working to generate diff report.
@nrmancuso , @rnveach , is there easy way to finish this PR ?
Let's start with message diff only for now. All further improvements are welcome as next step.
Ideally it would be good to have report file generated from validation to avoid our hack parsing of plugin output and dealing with OS differences of logging.
@mernst , is it possible to create report file from validation? and not rely on any output of plugin. We use output only because there is no report file of...
I would say, #5666 will allow users to have workaround. But issue with broken output is different issue, somebody else might run into it also if they start to deal...
@mernst, please let us know if we need to provide more details or something.
>we should take the check's description at face value and allow users to override the default as they see fit. Yes, this is my point. By default, Check doesn't target...