Copilot commented on code in PR #2969:
URL: https://github.com/apache/hugegraph/pull/2969#discussion_r2936742665
##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/profile/GraphsAPI.java:
##########
@@ -138,8 +142,11 @@ public Object get(@Context GraphManager manager,
@Produces(APPLICATION_JSON_WITH_CHARSET)
@RolesAllowed({"space"})
public void drop(@Context GraphManager manager,
+ @Parameter(description = "The graph space name")
@PathParam("graphspace") String graphSpace,
+ @Parameter(description = "The graph name")
@PathParam("name") String name,
+ @Parameter(description = "Confirmation message to drop
the graph, must be: " + CONFIRM_DROP)
Review Comment:
This `@Parameter` description line exceeds the project's 100-character
Checkstyle LineLength limit. Please wrap/shorten the description (e.g., move
the constant mention to a separate line or describe it without string
concatenation) to avoid build failures.
##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/profile/GraphsAPI.java:
##########
@@ -177,9 +185,13 @@ public Object reload(@Context GraphManager manager,
@Produces(APPLICATION_JSON_WITH_CHARSET)
@RolesAllowed({"space"})
public Object create(@Context GraphManager manager,
+ @Parameter(description = "The graph space name")
@PathParam("graphspace") String graphSpace,
+ @Parameter(description = "The graph name to create")
@PathParam("name") String name,
+ @Parameter(description = "The graph name to clone
from (optional)")
@QueryParam("clone_graph_name") String clone,
+ @Parameter(description = "The graph configuration
options including 'backend', 'serializer', 'store', and optionally
'description'")
Review Comment:
This `@Parameter` description line is longer than the 100-character
Checkstyle limit. Please wrap the description across lines or shorten it so the
file passes Checkstyle.
##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/schema/PropertyKeyAPI.java:
##########
@@ -100,9 +104,15 @@ public String create(@Context GraphManager manager,
"$action=property_key_write"})
@RedirectFilter.RedirectMasterRole
public String update(@Context GraphManager manager,
+ @Parameter(description = "The graph space name")
@PathParam("graphspace") String graphSpace,
+ @Parameter(description = "The graph name")
@PathParam("graph") String graph,
+ @Parameter(description = "The property key name")
@PathParam("name") String name,
+ @Parameter(description = "Action to perform: 'append'
to add new properties, " +
+ "'remove' to delete existing
properties, " +
+ "'clear' to clear OLAP
property data")
Review Comment:
The first line of this multi-line `@Parameter` description is likely to
exceed the 100-character Checkstyle LineLength limit due to indentation +
string content. Please reflow the annotation (e.g., break after "@Parameter("
or shorten the text) so each line stays within 100 characters.
--
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]
