Henning Schmiedehausen

Add `skipPoms` configuration parameter

2

Setting `true` will skip the `prepare-agent` and `prepare-agent-it` execution in POM (non-code) projects. This is desirable, however the default is `false` to be backwards compatible.

**Affects PMD Version:** 6.48 **Rule:** FinalFieldCouldBeStatic Please provide the rule name and a link to the rule documentation: https://pmd.github.io/latest/pmd_rules_java_design.html#finalfieldcouldbestatic **Description:** 6.48 flags objects that are only used for synchronization incorrectly...

NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE with `@CheckForNull` annotation

7

This code: ```java package testcode; import static java.util.Objects.requireNonNull; import java.util.function.Function; import javax.annotation.CheckForNull; public class SpotbugsTest { @CheckForNull public static Function get() { return k -> "nope"; } public static String...

Adds a decorator for Okhttp requests

1

This allows customization of all requests generated by the library through OkHttp, e.g. for tracking or adding custom headers.

fix mockito warnings

2

``` [INFO] Running org.jdbi.v3.core.mapper.TestEnums [MockitoHint] ArgumentBinderTest.testNamedEmpty (see javadoc for MockitoHint): [MockitoHint] 1. Unused -> at org.jdbi.v3.core.statement.ArgumentBinderTest.before(ArgumentBinderTest.java:46) [MockitoHint] 2. Unused -> at org.jdbi.v3.core.statement.ArgumentBinderTest.before(ArgumentBinderTest.java:47) [MockitoHint] ArgumentBinderTest.testNamedDeclaredButNotProvided (see javadoc for MockitoHint): [MockitoHint] 1....

all of these tests should pass. But they don't. They fail with exotic errors such as ``` org.jdbi.v3.core.statement.UnableToCreateStatementException: No argument factory registered for 'User:2' of qualified type org.jdbi.v3.core.argument.ObjectArgument [statement:"INSERT INTO...

modernizer is at odds with guava recommendations

5

According to https://code.google.com/p/guava-libraries/wiki/PreconditionsExplained, the guideline if using Guava in JDK7+ is --- snip --- We preferred rolling our own preconditions checks over e.g. the comparable utilities from Apache Commons for...

should support and empty rule set ("do nothing") configuration

1

It should be possible to "turn off" the default checks without having to specify any violations file. This would allow configuring the plugin in a stacked pom environment (having a...

There are a number of useful classes (MockFluentd) in the test code which are only accessible if the project builds and distributes test jars.

Adds "failOnWarning" tests

1

Support for "failOnWarning" was added in the 3.6.0 release. This is mostly unit and integration testing for the flag that I still had lying around. Would be a shame to...