Results 1611 comments of Roman Ivanov

Try update single class and send PR for review

Red pitest is hard blocker for merge, other 2 are failing even in master, they are blockers.

better to avoid deque in naming, same problem https://github.com/checkstyle/checkstyle/pull/12855#issuecomment-1474846212

Do we have same problem with this extension https://github.com/sevntu-checkstyle/sevntu.checkstyle ?

@robtimus, so it works on eclipsecs 9.3 and not working on eclipsecs 10.0 (next release after 9.3)? https://checkstyle.org/eclipse-cs/#!/releasenotes we changed min required Eclipse at 10.0.0 . f80470f0 Issue #340: change...

https://checkstyle.org/releasenotes.html#Release_10.5.0. >Allow 3rd party Check providers to group modules under custom parent module. Author: rnveach [#11644](https://github.com/checkstyle/checkstyle/issues/11644)

@rnveach , do you experience any issues with sevntu usage in Eclipse?

> mutatedClass com.puppycrawl.tools.checkstyle.Checker so you need to set property in Checker not in LineLength. Play with: ``` ```

lets try to move forward and fix code to make in-memory config to be correct (what we expect).

>Do you mean the config created by createConfiguration()? Yes. You can fix problem in this method in this PR, looks like it is first time we use config like this,...