Roman Ivanov
Roman Ivanov
plase try to use same version of as me. try to not use `-optimize-imports-only`
@Anmol202005 , can we finish this PR ?
while I waiting for laptop time to validate it... @Anmol202005 , can you make it as CI job ? same as we have idea inspection execution
@Anmol202005 , can you make CI job for it ? at checkstyle/.circleci/config.yml and may be `checkstyle/.ci/idea-format.sh` to simplify user experience to run it. @Pankraz76 , can you help us validate...
we targeting only `&&` and ` != null`. Any other methods that can check for null should be ignored. `data != null && (!Objects.equals(s, b) || data instanceof String)` should...
>Should I proceed creating a CI job for this configuration? yes, please. IDE agnostic update is not related at all, agnostic will never cover all formatting requirements, too much of...
we should write some method that loads Input file to list of Strings, get position of "//indent" from first line, validate that all other lines have it. should be not...
@voidCounter , thanks a lot for your HUGE work on alignment. please consider https://github.com/checkstyle/checkstyle/pull/17409#issuecomment-3146724011 as followup request to update more complicared cases. thanks a lot in advance!!
@Hariom-kr , here is latest state of this suppression list: https://github.com/checkstyle/checkstyle/blob/master/src/test/java/com/puppycrawl/tools/checkstyle/IndentationTrailingCommentsVerticalAlignmentTest.java#L43 please help us to finish this issue.
You need to show your changes in PR to let me understand what is a problem. Looks like you changed input that much that some violation popped up or disappeared....