gradle-baseline
gradle-baseline copied to clipboard
palantir
A set of Gradle plugins that configure default code quality tools for developers.
Addresses https://github.com/palantir/gradle-baseline/issues/1719 ## Before this PR The checkJavaVersions task directly accessed the project during execution, which is flagged in the configuration cache report: ...
###### _excavator_ is a bot for automating changes across repositories. Changes produced by the roomba/versions-props-latest check. To enable or disable this check, please contact the maintainers of Excavator.
When using compact constructors, record fields are not initialized until after the compact constructor has completed. It's very easy for devs to call an accessor in a compact constructor, thinking...
## Before this PR I mistakenly thought that com.palantir.baseline-exact-dependencies being applied (by default) meant that that `check` would run `checkUnusedDependencies`. Actually it does not, and repos wanting this behavior need...
###### _excavator_ is a bot for automating changes across repositories. Changes produced by the roomba/gradle8-pr check. To enable or disable this check, please contact the maintainers of Excavator.
## Before this PR Safety annotations were already properly propagated for `of` and `copyOf` static factories, but the stream collectors for multimaps were not properly propagating safety information. ## After...
## Before this PR Tests miss the case when a method is called on the built string, see https://github.com/palantir/gradle-baseline/issues/2812 ## After this PR Parens are now surround the append result,...
## What happened? Correctly spots a `StringBuilderConstantParameters` here, but suggests an incorrect fix: ```java new StringBuilder(foo) .append("_") .append(bar) .toString() .toLowerCase(); ``` Suggestion: ````java foo + "_" + bar.toLowerCase(); ```` ##...
## Before this PR Have to manually suppress DesignForExtension every time. ## After this PR ==COMMIT_MSG== Adding [TestFactory](https://junit.org/junit5/docs/5.4.1/api/org/junit/jupiter/api/TestFactory.html) (for DynamicTests) as part of the methods that ignore this annotation. ==COMMIT_MSG==...
## Before this PR ## After this PR ==COMMIT_MSG== ResourceIdentifierGetEqualsUsage rewrites code using `ResourceIdentifier#get*().equals(Object)` to `ResourceIdentifier#has*(String)` to avoid allocations. See https://github.com/palantir/resource-identifier/pull/423 ==COMMIT_MSG== ## Possible downsides?
