rnveach

Just notifying that this is still an issue 4 years later. Powermock is still blocked. If you run `verify` phase with Jacoco, you will have to do a clean after...

Only updating title as this is a global issue.

@bitwiseman Can you explain how `repo` is normally populated? I assume there is reflection going on here as there is no setter or assignment. https://github.com/hub4j/github-api/blob/e14ec3b3677760714cd096ad8157a3e6a6dded65/src/main/java/org/kohsuke/github/GHCommitPointer.java#L46

I think this check needs additional clarification. Around could mean, before or after, or could mean before and after. So whats expected for `{}`. `{}` or `{}` or `{}`...

Why not switch it to Javadoc's method of handling parsing failures and instead print a violation? They can then ignore the failures however they want (by suppression), completely or by...

Is there any reason we can't synch behavior between javadoc and java in this regards?

@romani Let me know when first post contains all final details I am approving.

> Unknown property name/value. It is looking for a string/null in the method call. Just inline the variable. https://github.com/sevntu-checkstyle/sevntu.checkstyle/blob/356ffe2ebf7fe69ec02a393a4e41bfb3c3074bc9/sevntu-checks/src/main/java/com/github/sevntu/checkstyle/checks/design/CheckstyleTestMakeupCheck.java#L322

That is correct. As I mentioned it only accepts strings/null and technically some paths. If it doesn't match those it will throw an exception. You technically shouldn't be using a...

The items in the `(` are part of the `ELIST` (expression list). The item in the first post is if `new c4()` should be in an expression token. ```` |...