imbajin commented on code in PR #415:
URL:
https://github.com/apache/incubator-hugegraph-doc/pull/415#discussion_r2444620787
##########
content/en/docs/quickstart/client/hugegraph-client.md:
##########
@@ -73,9 +73,10 @@ import org.apache.hugegraph.structure.gremlin.ResultSet;
public class SingleExample {
public static void main(String[] args) throws IOException {
- // If connect failed will throw a exception.
- HugeClient hugeClient = HugeClient.builder("http://localhost:8080",
+ HugeClient hugeClient = HugeClient.builder("http://127.0.0.1:8080",
+ "DEFAULT",
"hugegraph")
+ .configUser("admin", "admin")
Review Comment:
⚠️ **中英文配置不一致**: 英文版在这里添加了 `.configUser("admin", "admin")` 认证配置,但中文版没有添加。
建议:
1. 如果是演示不同场景(有/无认证),应在文档说明中注明
2. 否则建议中英文保持一致
3. 或在代码注释中说明使用场景,如:"// 当 Server 开启权限认证时配置"
##########
content/cn/docs/quickstart/client/hugegraph-client.md:
##########
@@ -348,12 +349,12 @@ public class BatchExample {
}
```
-### 4.4 运行 Example
+#### 4.4 运行 Example
Review Comment:
🧹 **标题层级错误**: 这里应该使用 `###` (三级标题)而不是 `####` (四级标题),以保持与 4.1-4.3 节的层级一致。
```suggestion
### 4.4 运行 Example
```
同样的问题也出现在 4.5 节。英文版也有相同问题需要修复。
--
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]