Results 1611 comments of Roman Ivanov

There is conflict, please rebase.

Please address Checker complains on missed annotation that something can be nullable. Pitest needs to be resolved by extra test or removal of unnecessary code `./.ci/pitest.sh "pitest-whitespace"` please run and...

@matrei , as you are author of Check idea, it should means that you have some project to apply it. Did you test your implementation on such projects ?

we need extend testing on real projects as we usually do. please look at https://github.com/checkstyle/checkstyle/pull/17233 , attention to PR description link, triggering testing is by comment like https://github.com/checkstyle/checkstyle/pull/17233#issuecomment-3341439148 github action...

Your draft PR is in right direction. Please do all test methods migration in separate PRs.

You need to use some Check in xml config that use severity ignore https://github.com/checkstyle/checkstyle/blob/359fa919d6b96ea334346459b77b7e54931dee65/src/test/java/com/puppycrawl/tools/checkstyle/XMLLoggerTest.java#L256

>since it would require an AuditEvent with null fileName. Could you please guide me? Yes, not possible if all other code works well, please add comment above test method to...

https://github.com/checkstyle/checkstyle/blob/62c038ddf9de15f2d9f4b5b9c2ca97ddac72c197/src/test/java/com/puppycrawl/tools/checkstyle/XMLLoggerTest.java#L258 You can try to run checkstyle on non compiled java Input and there will exception . >testNullOutputStreamOptions() testFinishLocalSetup() testCtorWithTwoParametersCloseStreamOptions() testCtorWithTwoParametersNoneStreamOptions() This likely can be only pure unit testing