[GitHub] [incubator-hugegraph-toolchain] imbajin commented on a diff in pull request #369: refact(client): adapt import & style with latest code

[GitBox]

imbajin commented on code in PR #369:
URL: 
https://github.com/apache/incubator-hugegraph-toolchain/pull/369#discussion_r1025263746


##########
hugegraph-client/src/test/java/org/apache/hugegraph/api/ApiTestSuite.java:
##########
@@ -52,51 +52,51 @@
 
 @RunWith(Suite.class)
 @Suite.SuiteClasses({
-    PropertyKeyApiTest.class,
-    VertexLabelApiTest.class,
-    EdgeLabelApiTest.class,
-    IndexLabelApiTest.class,
-    SchemaApiTest.class,
+        PropertyKeyApiTest.class,

Review Comment:
   how many spaces shall we keep in `@Annotation({p1, p2, p3})` ?  Is 4 spaces 
right?
   
   and we could unify the annotation style
   
   
![image](https://user-images.githubusercontent.com/17706099/202475046-a582c090-dc92-4b2d-8b6c-06e6a5f32cd4.png)
   



##########
hugegraph-client/src/test/java/org/apache/hugegraph/api/ApiTestSuite.java:
##########
@@ -52,51 +52,51 @@
 
 @RunWith(Suite.class)
 @Suite.SuiteClasses({
-    PropertyKeyApiTest.class,
-    VertexLabelApiTest.class,
-    EdgeLabelApiTest.class,
-    IndexLabelApiTest.class,
-    SchemaApiTest.class,
+        PropertyKeyApiTest.class,

Review Comment:
   @javeme how many spaces shall we keep in `@Annotation({p1, p2, p3})` ?  Is 4 
spaces right?
   
   and we could unify the annotation style
   
   
![image](https://user-images.githubusercontent.com/17706099/202475046-a582c090-dc92-4b2d-8b6c-06e6a5f32cd4.png)
   



-- 
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: dev-unsubscr...@hugegraph.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org