Roman Ivanov
Roman Ivanov
@vivek-0509 , please do diff regression for this Check. https://github.com/checkstyle/test-configs/tree/main/MissingOverride/all-examples-in-one#option-1
@Zopsss , please help me review this google style update
@vivek-0509 , sorry that we wasted your time to craft this PR. I hope to see most of this code in new PR for new Check.
@HamishHare , > [ERROR] /var/tmp/IndentationIssueDemo.java:19:5: 'method def' child has incorrect indentation level 4, expected level should be 16. [Indentation] do you mean that message of violation is misleading ? `'method...
yes, lets fix message it should be `+' has incorrect indentation level 4, expected level should be 16. [Indentation]`, same as for line 10.
@Jiya873 , Indenation is most complicated module in our project, if you find it complicated to fix, feel free to abandon it and help us with some other issue.
@kalpadiptyaroy, please find time to finish this PR
Look like PR is too complicated. I am closing it to let someone more experienced to continue work. @kalpadiptyaroy , please be welcome with contribution to other issues. Sorry that...
> Checkstlyle does not use compiled bytecode (right?), yes > we can see that checkstyleMain and checkstyleTest require compileJava, classes, processResources, etc... to run first: checkstyle always should run after...
> but the job peaks at ~12GB memory usage. do we have leak in Checkstyle ? It might be possible. We use openjdk as target project for regression testing (without...