Open nakulj opened 3 months ago
Thanks for your interest in palantir/gradle-baseline, @nakulj! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.
Type
Description
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, ensuring that the post-build method call is applied to the entire built string, not just the last element.
Possible downsides?
Adds parens even when unnecessary. These can be caught and fixed by UnnecessaryParentheses, but ideally they'd only be added if necessary.