Results 498 comments of rnveach

Have you tried the other items before the "fallback" one? If the others are not working, please explain why.

> Though running mvn package results in build failure: This is the command we run in CI. If you still have issues I would need to know what the failure...

Quick google on error message says: > GitHub's Permission denied (publickey) error is usually caused by one of the following three issues: You have used an incorrect email address in...

@ZuevKirill95 You have to rebase on the latest master and remove all merge commits. > Maybe remove this commit? Same changes in https://github.com/checkstyle/checkstyle/pull/12039 I am not understanding why we have...

I thought my understanding about 11529 was that it was to be closed in favor of this PR as we were using that as a test area to identify the...

I am marking this as blocked since I don't think its safe to merge in `Locale.setDefault(Locale.ENGLISH);`. We should be thinking about something like #12104 to resolve the underlying issue. If...

We have import statement which has method name on the end. Is there a reason we cannot look for a method call using that method name in the Java file?...

> The problem comes when we have two or more imported methods that have the same name . Yes, this is a limitation of going by the source but is...

@romani The issue I found during my own testing is we can't determine if all methods are used, only 1, ~or none~. It is possible that both methods can be...

@stoyanK7 Yes this is a limitation of Checkstyle, but we should update the documentation to show this and this is why we cannot determine which is used/unused in this circumstance.