psxjoy commented on code in PR #943:
URL: https://github.com/apache/fesod/pull/943#discussion_r3542225791
##########
fesod-sheet/src/main/java/org/apache/fesod/sheet/metadata/AbstractParameterBuilder.java:
##########
@@ -50,6 +52,18 @@ public T head(List<List<String>> head) {
return self();
}
+ /**
+ * You can only choose one of the ({@link #head(List)} or {@link
#head(Consumer)}) and {@link #head(Class)}
Review Comment:
same above
##########
fesod-sheet/src/main/java/org/apache/fesod/sheet/metadata/AbstractParameterBuilder.java:
##########
@@ -68,7 +82,7 @@ private List<List<String>>
toMutableListIfNecessary(List<List<String>> head) {
}
/**
- * You can only choose one of the {@link #head(List)} and {@link
#head(Class)}
+ * You can only choose one of the ({@link #head(List)} or {@link
#head(Consumer)}) and {@link #head(Class)}
Review Comment:
same above
##########
fesod-sheet/src/main/java/org/apache/fesod/sheet/metadata/AbstractParameterBuilder.java:
##########
@@ -40,8 +41,9 @@
*/
public abstract class AbstractParameterBuilder<T extends
AbstractParameterBuilder, C extends BasicParameter> {
/**
- * You can only choose one of the {@link #head(List)} and {@link
#head(Class)}
+ * You can only choose one of the ({@link #head(List)} or {@link
#head(Consumer)}) and {@link #head(Class)}
Review Comment:
Since this remark should be modify,better not using 'You' .
既然这个注释已经修改了，我觉得原始的注释有点太口语化了，最好不要用“你”这种词汇？
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
