Skip to content

docs: fix 错别字 in 2-kprobe-unlink Chinese README (分别时 -> 分别是) - #229

Open
yunwei37 wants to merge 1 commit into
mainfrom
fix/zh-kprobe-typo
Open

yunwei37 wants to merge 1 commit into
mainfrom
fix/zh-kprobe-typo

Conversation

@yunwei37

Copy link
Copy Markdown
Member

Reported in #228.

src/2-kprobe-unlink/README.zh.md:13 reads "kprobes 技术包括的3种探测手段分别时 kprobe、jprobe 和 kretprobe", where 分别时 should be 分别是. The English README states the same sentence correctly as "three detection methods: kprobe, jprobe, and kretprobe", so the Chinese text is the one at fault.

This is the only occurrence of 分别时 in the repository, so the fix is the single character on that line — no other content touched.

分别时 -> 分别是, the three probe types are kprobe, jprobe and kretprobe.
Reported in #228.
Copilot AI lite review requested due to automatic review settings September 24, 2026 12:46

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@yunwei37

Copy link
Copy Markdown
Member Author

Head cd5879bc is the one-line change reported in #228 and it is fully green: build (x3), trigger, docs, GitGuardian, CodeFactor, mergeStateStatus=CLEAN, no review threads.

Note on review: this branch was authored by the patrol, so a patrol approval of it would be self-review rather than independent validation. The head therefore still needs an approving review from someone else. This repository is well above 500 stars, so the merge itself is the maintainer's call.

For the record, the first CI run on this head failed on a transient runner TLS error downloading ecc (Unable to establish SSL connection from release-assets.githubusercontent.com), not on the change; the download succeeds on retry and the rerun passed.

AI-generated response; a maintainer will review and follow up later

This branch has not been deployed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants