Roman Ivanov
Roman Ivanov
@vivek-0509 , your understanding is good, my concern on interfaces is cleared. >Do we strictly need to exclude records with an implements clause? only for records and it accessor to...
to approve this issue we need to see behavior of javadoc tool that generates html content. Example - https://github.com/checkstyle/checkstyle/issues/13048#issuecomment-1546663328 We always look at javadoc tool for expected behavior.
Everyone in world is welcome to send Pull Request. Sorry for long approval.
@Zopsss , please help to review this PR, I have only minor request. I am good to merge this as soon as it is fixed.
Please rebase to most latest code , CI is red as reminder to you. Please follow.
This issue is not easy for new contributors Please [read](https://github.com/checkstyle/checkstyle/blob/master/.github/CONTRIBUTING.md#getting-started) and watch videos at [Starting_Development](https://checkstyle.org/beginning_development.html#Starting_Development). No assignments, contributors just make comment "I am in ....'" and can start working
You subconsciously choose most complicated modules :). I recommend to finish other pitest first. Indentation is very problematic module, that has numerous defects.
I removed approved label to let people to focus on other issues first. As this Check needs special attention and knowledge.
Please share full CLI output for javadoc as all others are doing
All is green now, but please share CLI output in PR description. And in javadoc we keep symbols that shows tre structure