Skip to content

Commit f99a5ad

Browse files
Apply suggestions from code review
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
1 parent d974b9c commit f99a5ad

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/main/java/org/openrewrite/java/migrate/lombok/FieldAnnotator.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
import org.openrewrite.java.tree.JavaType;
2828

2929
import java.util.List;
30-
import java.util.stream.Collectors;
30+
import static java.util.stream.Collectors.joining;
3131

3232
import static java.util.Comparator.comparing;
3333
import static lombok.AccessLevel.PUBLIC;
@@ -59,8 +59,7 @@ public J.VariableDeclarations visitVariableDeclarations(J.VariableDeclarations m
5959
if (onMethodAnnotations.isEmpty()) {
6060
suffix = valueArg.isEmpty() ? "" : String.format("(%s)", valueArg);
6161
} else {
62-
String onMethodArg = String.format("onMethod_ = {%s}", onMethodAnnotations.stream().map(J.Annotation::toString).collect(Collectors.joining(",")));
63-
suffix = valueArg.isEmpty() ? String.format("(%s)", onMethodArg) : String.format("(value = %s, %s)", valueArg, onMethodArg);
62+
String onMethodArg = String.format("onMethod_ = {%s}", onMethodAnnotations.stream().map(J.Annotation::toString).collect(joining(",")));
6463
}
6564

6665
return JavaTemplate.builder("@" + annotation.getSimpleName() + suffix)

0 commit comments

Comments
 (0)