Copilot commented on code in PR #2966:
URL: https://github.com/apache/hugegraph/pull/2966#discussion_r2930146640
##########
hugegraph-server/hugegraph-test/src/main/java/org/apache/hugegraph/api/BaseApiTest.java:
##########
@@ -519,19 +510,20 @@ protected static void initOrClear() {
// isn't hstore
BaseApiTest.clearData();
}
- } else {
+ }
+ else {
Review Comment:
The brace style `}\nelse {` (else on a new line) deviates from the
established codebase convention of `} else {` (else on the same line as the
closing brace). This is consistently used throughout the codebase (e.g.,
`API.java:239`, `GraphSpaceAPI.java:129`, and dozens of other files). The same
applies to the `}\ncatch` on line 537-538. These appear to be auto-formatter
artifacts and should be reverted to match the codebase convention.
##########
hugegraph-server/hugegraph-test/src/main/java/org/apache/hugegraph/api/BaseApiTest.java:
##########
@@ -542,14 +534,15 @@ protected static <T> List<T> readList(String content,
JavaType type = MAPPER.getTypeFactory()
.constructParametricType(List.class, clazz);
return MAPPER.readValue(element.toString(), type);
- } catch (IOException e) {
+ }
+ catch (IOException e) {
Review Comment:
Same issue: `}\ncatch` deviates from the codebase convention of `} catch {`
on the same line. See the same convention established throughout the codebase
(e.g., `API.java:239`, `GraphsAPI.java:113`).
--
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]
