Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

at: update French and Chinese translation #14653

Merged
merged 12 commits into from
Nov 17, 2024

Conversation

sebastiaanspeck
Copy link
Member

Ref: #14220

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The page(s) follow the style guide.
  • The PR title conforms to the recommended templates.

@github-actions github-actions bot added the translation Translate pages from one language to another. label Nov 5, 2024
@sebastiaanspeck
Copy link
Member Author

Feel free to update the /common/at.md in this PR, which is outdated in French and Chinese.

@sebastiaanspeck sebastiaanspeck changed the title at: remove the Linux variant at: update French and Chinese translation Nov 5, 2024
pages.fr/common/at.md Outdated Show resolved Hide resolved
pages.fr/common/at.md Outdated Show resolved Hide resolved
noraj
noraj previously requested changes Nov 5, 2024
pages.fr/common/at.md Outdated Show resolved Hide resolved
pages.fr/common/at.md Outdated Show resolved Hide resolved
pages.zh/common/at.md Outdated Show resolved Hide resolved
BananaaaKING

This comment was marked as duplicate.

pages.zh/common/at.md Outdated Show resolved Hide resolved
pages.zh/common/at.md Outdated Show resolved Hide resolved
sebastiaanspeck and others added 4 commits November 6, 2024 10:10
Co-authored-by: Alexandre ZANNI <[email protected]>
Co-authored-by: Alexandre ZANNI <[email protected]>
Co-authored-by: Jin <[email protected]>
Co-authored-by: Jin <[email protected]>
@LiLittleCat
Copy link
Collaborator

LiLittleCat commented Nov 6, 2024

BTW, @sebastiaanspeck @blueskyson, do we need to strictly keep stdin in English instead of translating it to 标准输入? The style guide suggests avoiding translations for certain technical terms:

在命令描述中,如果出现了技术性的专有名词,请用 `反引号` 括起来:
- 路径,例如 `package.json``/etc/package.json`
- 扩展名,例如 `.dll`
- 命令,例如 `ls`
- 标准流:`stdout``stdin``stderr`**不要**使用完整的名称(例如标准输出)。

I noticed some existing translations don’t fully follow this guideline. If we decide to enforce this strictly, I can review and adjust any inconsistent translations.

pages.fr/common/at.md Outdated Show resolved Hide resolved
Co-authored-by: Nicolas Kosinski <[email protected]>
@sebastiaanspeck
Copy link
Member Author

BTW, @sebastiaanspeck @blueskyson, do we need to strictly keep stdin in English instead of translating it to 标准输入? The style guide suggests avoiding translations for certain technical terms:

在命令描述中,如果出现了技术性的专有名词,请用 `反引号` 括起来:
- 路径,例如 `package.json``/etc/package.json`
- 扩展名,例如 `.dll`
- 命令,例如 `ls`
- 标准流:`stdout``stdin``stderr`**不要**使用完整的名称(例如标准输出)。

I noticed some existing translations don’t fully follow this guideline. If we decide to enforce this strictly, I can review and adjust any inconsistent translations.

If it is in the guideline, we should follow it. If you find translations who do not follow, feel free to open a PR.

pages.zh/common/at.md Outdated Show resolved Hide resolved
pages.zh/common/at.md Outdated Show resolved Hide resolved
pages.zh/common/at.md Show resolved Hide resolved
Copy link
Collaborator

@nicokosi nicokosi left a comment

Choose a reason for hiding this comment

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

I "soft-approve": French version is OK for me but I can't approve the Chinese part. 😇

Co-authored-by: Jin <[email protected]>
@sebastiaanspeck
Copy link
Member Author

@Nico385412 @nicokosi @noraj could you please look at this PR? We are waiting for your review.

Copy link
Collaborator

@noraj noraj left a comment

Choose a reason for hiding this comment

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

LGTM now for the French part

@sebastiaanspeck sebastiaanspeck merged commit 4486e86 into tldr-pages:main Nov 17, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation Translate pages from one language to another.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants