Roman Ivanov

migration is merged

I detached pullRequest builds  with hope it will stop execution of TC on each PR

Please provide regression testing on real projects. See details at https://github.com/checkstyle/contribution/tree/master/checkstyle-tester#executing-generation all is automated, you just need to provide config and file with target projects. You can find bunch of...

please add https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/bbfd2c4_2022092552/reports/diff/openjdk17/index.html#A5 to Input, it is pretty weird. add https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/bbfd2c4_2022092552/reports/diff/openjdk17/index.html#A11 to Inputs, as example of new variable wihtout assignment of it. Idea behind extension of tests from regression testing...

All my points are addressed. @ZuevKirill95 , thanks a lot , sorry for long pause to reply.

GitHub, rebase

@ZuevKirill95 , please rebase your code on most latest master. Auto rebase is failed https://github.com/checkstyle/checkstyle/actions/runs/3124120471/jobs/5067208282#step:6:8

I recommend to not mess with hashcode/equals (human is always error prone on this contract) and make special method `compareTo` or somthing like this and use it instead. We do...

>As we use Sets/ TreeSets/ Lists, we must make sure that equals and hashcode are overridden Nobody has to override such methods if objects are immutable or you do not...

Instances of two violations are not same, but we can consider them as same if that is unstable and we want it. But they are not same and equals should...