Results 1611 comments of Roman Ivanov

>however I would expect google-java-format to be the reference implementation) google-java-format appears [year](https://github.com/google/google-java-format/releases?page=3) after checkstyle [provided](https://github.com/checkstyle/checkstyle/commits/c17bf06af9334ea1f6096868e591e6ee62774540/google_checks.xml?browsing_rename_history=true&new_path=src/main/resources/google_checks.xml&original_branch=master) style validation. But now a day better to be starting looking on each other....

Please create issue on Google style to clarify it, if they update doc as google-java-format doing in some version (latest for today) we can change checkstyle config and Check to...

@cushon , thanks a lot for detailed report. yes, checkstyle style guide coverage is outdated .... we are stuck in 2018 year https://checkstyle.org/styleguides/google-java-style-20180523/javaguide.html there were no such exception for case...

Please activate: [AmbiguousJsonCreator](https://error-prone.picnic.tech/bugpatterns/AmbiguousJsonCreator/) [AssertJIsNull](https://error-prone.picnic.tech/bugpatterns/AssertJIsNull/) [AutowiredConstructor](https://error-prone.picnic.tech/bugpatterns/AutowiredConstructor/) [CanonicalAnnotationSyntax](https://error-prone.picnic.tech/bugpatterns/CanonicalAnnotationSyntax/) [CollectorMutability](https://error-prone.picnic.tech/bugpatterns/CollectorMutability/) --------- [DirectReturn](https://error-prone.picnic.tech/bugpatterns/DirectReturn/) we can try in separate PR, to see a scope of violations, I remember in some cases it was benefitials to...

@rickie, please send each activation in separate PR, AND isf there is bunch of udpates required, it is completely OK to split fixes( before activation) in separate PRs. Some updates...

This is not good for junior contributors as it might be controversial and tricky. But you are not a junior, we are welcome with PR

@rickie , thanks a lot for your hard work!!!

the same NPE even on old container that we used for several years "checkstyle/jdk-11-groovy-git-mvn:11.0.13__3.0.9__2.25.1__3.6.3" so regression is in FieldItemSummary This is very strange we have execution of spotbugs from in...

As I stated above, we don't know a way to reproduce. Might be very special nuance of jdk version jdk11.0.20.1.