Repository navigation
docs: Translate the Chinese version of the document - #869
Conversation
Code Review SummaryStatus: No Issues Found | Recommendation: Merge OverviewThis PR is a documentation-only translation task that converts English documentation to Chinese (Simplified) for 14 files in
Files Reviewed (14 files)
Notes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request provides comprehensive updates and translations for the Chinese (zh-CN) documentation, covering modules such as the A2A server, API reference, and system architecture. It also includes details on new features like Cloudflare Quick Tunnel and updates the CLI tools configuration guide. The review feedback correctly identifies syntax errors in the VM deployment guide caused by the use of smart quotes in shell commands and Nginx configurations, offering specific code suggestions to ensure compatibility and correct execution.
| @@ -92,15 +92,15 @@ mkdir -p /opt/omniroute | |||
|
|
|||
| ```bash | |||
| cat > /opt/omniroute/.env << ‘EOF’ | |||
There was a problem hiding this comment.
The heredoc delimiter uses smart quotes (‘, ’). This may not be interpreted correctly by all shells. Use standard single quotes (').
| cat > /opt/omniroute/.env << ‘EOF’ | |
| cat > /opt/omniroute/.env << 'EOF' |
| @@ -174,7 +174,7 @@ chmod 600 /etc/nginx/ssl/origin.key | |||
|
|
|||
| ```bash | |||
| cat > /etc/nginx/sites-available/omniroute << ‘NGINX’ | |||
There was a problem hiding this comment.
The heredoc delimiter uses smart quotes (‘, ’). Use standard single quotes (').
| cat > /etc/nginx/sites-available/omniroute << ‘NGINX’ | |
| cat > /etc/nginx/sites-available/omniroute << 'NGINX' |
| # WebSocket 支持 | ||
| proxy_http_version 1.1; | ||
| proxy_set_header Upgrade $http_upgrade; | ||
| proxy_set_header Connection “upgrade”; |
There was a problem hiding this comment.
The Nginx configuration uses smart quotes (“, ”). This will cause a syntax error when Nginx tries to load the configuration. Use standard double quotes (").
| proxy_set_header Connection “upgrade”; | |
| proxy_set_header Connection "upgrade"; |
| @@ -323,7 +323,7 @@ docker start omniroute | |||
|
|
|||
| ```bash | |||
| cat > /etc/nginx/cloudflare-ips.conf << ‘CF’ | |||
There was a problem hiding this comment.
The heredoc delimiter uses smart quotes (‘, ’). Use standard single quotes (').
| cat > /etc/nginx/cloudflare-ips.conf << ‘CF’ | |
| cat > /etc/nginx/cloudflare-ips.conf << 'CF' |
|
@diegosouzapw Did something go wrong during the merge? Why are the Chinese documents still in English on the latest branch? |
No description provided.