enable all errorprone checks by default and resolve their nit warnings · ben-manes/caffeine@3b9e723

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/AbstractJCacheTest.java#L50

[MultiVariableDeclaration] Variable declarations should declare only one variable

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/AbstractJCacheTest.java#L51

[MultiVariableDeclaration] Variable declarations should declare only one variable

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/AbstractJCacheTest.java#L52

[MultiVariableDeclaration] Variable declarations should declare only one variable

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/integration/CacheWriterTest.java#L45

[FieldCanBeFinal] This field is only assigned during initialization; consider making it final

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/management/JCacheStatisticsMXBeanTest.java#L29

[Varifier] Consider using `var` here to avoid boilerplate.

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/copy/JavaSerializationCopierTest.java#L156

[MethodCanBeStatic] A private method that does not reference the enclosing instance can be static

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/copy/JavaSerializationCopierTest.java#L126

[Varifier] Consider using `var` here to avoid boilerplate.

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/spi/CaffeineCachingProviderTest.java#L130

[MethodCanBeStatic] A private method that does not reference the enclosing instance can be static

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/configuration/JCacheConfigurationTest.java#L81

[MethodCanBeStatic] A private method that does not reference the enclosing instance can be static

Build: jcache/src/test/java/com/github/benmanes/caffeine/jcache/CacheProxyTest.java#L51

[FieldCanBeFinal] This field is only assigned during initialization; consider making it final

Provide feedback