Results 510 comments of rnveach

Documentation update was merged

I think this will connect into an issue with test fields and weither they are allowed to be mutable or not. https://github.com/checkstyle/checkstyle/pull/12053#discussion_r953876062 > ViolationTest has a static final Locale which...

> How can I access the source code of https://checkstyle.sourceforge.io/anttask.html https://github.com/checkstyle/checkstyle/blob/master/src/xdocs/anttask.xml.vm > I need to Make the changes that you think are needed and we will review. It should cover...

Dependabot will not do this for us? We shouldn't have a conflict due to step 3 but... Maybe we should file an improvement request with dependabot...

Splitting this up may be a bit difficult because filters need access to the localized text displayed since it can suppress on the displayed text. I wouldn't consider Filters an...

This issue was not the main subject of the PR merged

```` PS M:\checkstyleWorkspaceEclipse\checkstyle> groovy -version WARNING: An illegal reflective access operation has occurred WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/C:/groovy-2.5.18/lib/groovy-2.5.18.jar) to method java.lang.Object.finalize() WARNING: Please consider reporting this to the...

> I sent a fix for proper equals and hashCode contract in https://github.com/checkstyle/checkstyle/pull/12223, please check after applying that patch. No real difference on my end.

> Please try with the version and distribution used in CI So have we confirmed this tool is directly tied to a specific compiler and if something changes in our...