Results 1611 comments of Roman Ivanov

if user is alread in jdk17, he can forbit usage of token by https://checkstyle.org/config_coding.html#IllegalToken so easy. ``` ``` instead of (that do the same goal, violate modifier irrespective of jdk):...

User can override message to make it same in wording

It is good Check to do this - no disagreement. We never used tokens for such behavior and this weird for me. It is more logical to make "jdkVersion" field,...

Jdk version in TreeWalker is too much. If we make parser be able to disable some features, it will force us to test a lot with different versions of jdk....

@timurt , what is a reason of closing ? I still see https://github.com/checkstyle/checkstyle/blob/85ddafbbf2c6f1bb85ca928f5fd998b5447053f2/.ci/pitest.sh#L61-L64

@ilopmar , thanks a for sharing idea. >In Java it's considered a good practice to sort all imports alphabetically. to be precise: do grouping and sort inside groups alphabetically. User...

@nmancus1 , @rnveach , @strkkk , @pbludov , lets think/vote/re-consider on our decision to avoid extra suppresion/ignore properties. We use to make this decision as we had no time to...

usually we resolved such cases, by example in documentation (xdoc) on how to handle frequent edge cases, so googling usually helps to find solution.

we did not make it like this few years ago, as implementation of Xpath was not mature enough, we still have items to finish to make it for all Checks....

Deprecation is long for us and painful process for users. Better to keep two options and let them work by same code inside. Xpath is too brutal to user, it...