Skip to content

docs(zh): proofread README_zh.md — fix two confirmed errors - #982

Closed
t957095 wants to merge 1 commit into
libredb:mainfrom
t957095:docs/zh-proofread
Closed

t957095 wants to merge 1 commit into
libredb:mainfrom
t957095:docs/zh-proofread

Conversation

@t957095

@t957095 t957095 commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Full native-speaker proofread of README_zh.md against the English original, per #974. The file is in genuinely good shape overall; two confirmed errors fixed:

  1. Trino row: 连接所固定 catalog 的 information_schema schema 树 was missing 的, leaving a broken clause. Source: information_schema schema tree for the catalog the connection pins.
  2. Agent-mode limits: 运行还没有建立 mistranslates before a run is opened — the run is never created, not merely 'not yet established'. Fixed to 根本不会创建运行.

Everything else checked out as accurate and idiomatic, including the engine table, the agent pipeline description, and the distribution section.

Part of #974

- Trino row: '连接所固定 catalog' was missing 的, breaking the clause
  ('schema tree for the catalog the connection pins').
- Agent-mode limits: '运行还没有建立' mistranslated 'before a run is
  opened' — the run is never created at all.

Full read-through as a native speaker; the rest of the file reads
accurately and idiomatically against the English original.

Part of libredb#974
@cevheri cevheri added the documentation Improvements or additions to documentation label Sep 19, 2026
@cevheri cevheri linked an issue Sep 19, 2026 that may be closed by this pull request
@cevheri

cevheri commented Sep 19, 2026

Copy link
Copy Markdown
Member

thanks @t957095 and welcome

@cevheri cevheri added the loop:needs-moderator-action Flagged by the maintainer loop: suspicious content or a decision only a human can make label Sep 19, 2026

@InnoxCodes InnoxCodes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked both fixes against the English README. The Trino row now reads correctly (连接所固定的 catalog), and the English says the workflow is refused "before a run is opened", so 根本不会创建运行 is the better wording. node scripts/readme-check.mjs passes on this branch.

One note: #1005 says it includes both of these fixes and adds more, so the two PRs overlap and only one of them needs to be merged.

@t957095

t957095 commented Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

Closing this per the review below: #1005 includes both of these fixes and adds more, so only one of the two PRs needs to merge. Thanks @cevheri for the welcome and @InnoxCodes for the review.

@t957095 t957095 closed this Sep 19, 2026
@cevheri

cevheri commented Sep 20, 2026

Copy link
Copy Markdown
Member

Thanks, just a quick reminder: could you leave a comment so I can assign this issue to you.
issue number: #974

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation loop:needs-moderator-action Flagged by the maintainer loop: suspicious content or a decision only a human can make

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proofread README_zh.md: no Chinese speaker has ever reviewed it

3 participants