Results 508 comments of rnveach

This doesn't tell me why we can remove the code and not have an issue. Adding a test case with no insight into the changed code is just another form...

For unnamed variables: One check needs to be Indentation. It has to execute on all tokens otherwise things are "hidden" from it. IllegalToken should support it, if it doesn't already....

> we need to update this [example](https://checkstyle.org/checks/javadoc/javadocstyle.html#:~:text=*%20Second%20line%20of%20description%20//%20violation%2C%20the%20sentence%20must%20end%20with%20a%20proper%20punctuation) in the documentation. Can you fix the link to the example so it is using our linking for examples. When I bring up...

I was told CI should be fixed, so it shouldn't fail except for failures and random internet outages.

Issue was never approved, and I am not really seeing a clear definition of issue when looking, and PR was closed. I am hesitant to see why this issue would...

@romani > We can make violation message to show That will not work for me. As I reported, Eclipse likes to add these spaces to the Javadoc by default and...

Also please look over my verbage in the other PR to better explain what I am looking for.

I cannot approved because of https://github.com/checkstyle/checkstyle/pull/14181#pullrequestreview-1798340029 .

> I thought https://github.com/checkstyle/checkstyle/issues/14178 explains all. It explains more after your update. > all end up to null, we do not want null This goes back to my question `why...