Results 1611 comments of Roman Ivanov

@Zopsss , > Below is an example file: please update example to show where violation is expected by comments, same as we do in our Inputs for test and Example...

@Zopsss, please move all to issue description to have single place with all details and examples. I will remove all comments as we finalize discussion, to avoid confusion from next...

Please update examples to put violations on constructors that are happening after nonctor. Check should violate only constructors.

> Example5 { // violation, compact constructor should be first in order Why violation? ------ Name of module become a confusion. As we do not care about order and just...

@relentless-pursuit , please don't open new PRs, let's finish hanging PRs like this

I thought parent pom was required to be able to deploy artifacts to maven central. Is it not an issue anymore?

we reverted changes as it caused problems for release - https://github.com/checkstyle/checkstyle/actions/runs/8066068513/job/22033791002#step:6:399 ``` [INFO] Performing remote staging... [INFO] [INFO] * Remote staging into staging profile ID "aeb23e7bacef6f" [INFO] * Created staging...

@MANISH-K-07 , > Is there anything I could do to help ? we still need to investigate and fix this problem.

> * Missing: no javadoc jar found in folder '/com/puppycrawl/tools/checkstyle/10.14.0' looks like parent pom defined some config for javadoc generation, and we need to restore it.

@GitToasterhub , we are not sure how exactly to change code. Our primary care is that behavior of Check is correct and it validates declared cases correctly. As all works...