Skip to content

Conversation

youmeim
Copy link
Contributor

@youmeim youmeim commented Jun 23, 2025

Reopened PR, which is originated from #49695 .

Description

This PR contains a small modification on content/ja/docs/contribute/localization.md, which provides more clear translation workflow to support decisions by contributors.

Reviewed at once by @nasa9084 and @Okabe-Junya .
(Thanks for your kindly mention this, and sorry for bothering you!)

This PR is just related to Japanese localization, so Japanese comments or discussions are prefered/reccomended.

Issue

Closes: #49285

/area localization
/language ja

@k8s-ci-robot k8s-ci-robot added area/localization General issues or PRs related to localization language/ja Issues or PRs related to Japanese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Jun 23, 2025
@k8s-ci-robot k8s-ci-robot requested review from b1gb4by and bells17 June 23, 2025 14:51
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jun 23, 2025
Copy link

netlify bot commented Jun 23, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 648585f
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/686359017549e40008fd7e61
😎 Deploy Preview https://deploy-preview-51357--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@youmeim
Copy link
Contributor Author

youmeim commented Jun 23, 2025

とりあえず立てておきます。お手数をおかけして大変申し訳ありません。
最新の議論のチェック&追随はできていませんが、できる範囲で見直しておきます。

preview: https://deploy-preview-51357--kubernetes-io-main-staging.netlify.app/ja/docs/contribute/localization/

このPRの概要はこちらで議論した通りのものです。
また、日本語ローカリゼーションガイドについて、以前の議論で挙がっていた修正内容が含まれています。

また、マージに関しては標準化用語集の翻訳が完遂されていることが望ましいとの指摘がありました。

from #49285 (comment)
標準化用語集( https://kubernetes.io/ja/docs/reference/glossary/?all=true ) に記載の用語は、その表現に準拠させる
ただしその場合は、 https://kubernetes.io/ja/docs/reference/glossary/?all=true の翻訳を完了させることがガイドライン更新の必要条件になると思います

レビューにご参考頂ければと思います。

Copy link
Member

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

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

ありがとうございます、数点コメントさせていただきました

@Okabe-Junya
Copy link
Member

/cc t-inu

@k8s-ci-robot k8s-ci-robot requested a review from t-inu June 23, 2025 16:12
@Okabe-Junya
Copy link
Member

/hold

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 23, 2025
@nasa9084
Copy link
Member

LGTM, other than okabe-san's comments, thank you~!

@Okabe-Junya
Copy link
Member

/label tide/merge-method-squash

@k8s-ci-robot k8s-ci-robot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Jun 24, 2025
@Okabe-Junya
Copy link
Member

#51357 (comment) の確認だけお願いします、それ以外はLGTMです

Copy link

linux-foundation-easycla bot commented Jun 25, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@k8s-ci-robot k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jun 25, 2025
@youmeim youmeim force-pushed the 49285-modify-localization-guide-ja branch from 4c63e61 to 0d44ec6 Compare June 25, 2025 14:24
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Jun 25, 2025
@youmeim
Copy link
Contributor Author

youmeim commented Jun 25, 2025

t-inu Okabe-Junya nasa9084 レビューありがとうございます。反映しました。#51357 (comment)

Copy link
Member

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

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

ありがとうございます、nit commentを残しましたがLGTMです

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 25, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 25ed2dcc748fe8dc0e17fa87d6ef55bbcfd0ac22

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 27, 2025
@k8s-ci-robot k8s-ci-robot requested a review from Okabe-Junya June 27, 2025 14:40
Copy link
Member

@t-inu t-inu left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 30, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 6dd39e550c48457a91f58a10a6c1a0639a9d5766

Copy link
Member

@Okabe-Junya Okabe-Junya left a comment

Choose a reason for hiding this comment

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

@youmeim

repo全体でNetlifyのbuildに失敗する問題が発生していました。upstream/mainに修正が入って解消されたので、upstream/main でこのPRをrebaseしていただきたいです

/assign
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Okabe-Junya

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 30, 2025
youmeim and others added 5 commits July 1, 2025 12:39
Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
@youmeim youmeim force-pushed the 49285-modify-localization-guide-ja branch from 977153a to 648585f Compare July 1, 2025 03:41
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 1, 2025
@k8s-ci-robot k8s-ci-robot requested review from Okabe-Junya and t-inu July 1, 2025 03:41
@youmeim
Copy link
Contributor Author

youmeim commented Jul 1, 2025

@youmeim

repo全体でNetlifyのbuildに失敗する問題が発生していました。upstream/mainに修正が入って解消されたので、upstream/main でこのPRをrebaseしていただきたいです

/assign
/approve

@Okabe-Junya ありがとうございます。
対応しました。

今更ですが、自分のレビューコメントがpendingとなっていて、未ログイン時に自分自身のコメントが表示されないことに気づきました。
organizationのメンバーは見えているものと思っていますが、間違いないでしょうか。。。

レビュー頂いた際のレスポンスのうち、きちんと残した方が良いものについては、メインスレッドで議論した方が良さそうだなと思いました。

@t-inu
Copy link
Member

t-inu commented Jul 1, 2025

Pending状態のコメントは、本人以外は誰からも見えないと思います。
Files changedの画面で、右上にあるReview changesから、Submit reviewボタンを押すことで公開されます。

@youmeim
Copy link
Contributor Author

youmeim commented Jul 1, 2025

Pending状態のコメントは、本人以外は誰からも見えないと思います。 Files changedの画面で、右上にあるReview changesから、Submit reviewボタンを押すことで公開されます。

そうなのですね。
ということは、レビューに直接関連づけてコメントする際は、Conversation内ではなくFile changedタブ側で作業したほうが良さそうですね...

ご教示ありがとうございます。

Copy link
Member

@t-inu t-inu left a comment

Choose a reason for hiding this comment

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

/lgtm
/unhold

@k8s-ci-robot k8s-ci-robot added lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Jul 1, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 5652f14427d154ccc344df0349bba53276286e7b

@k8s-ci-robot k8s-ci-robot merged commit d790240 into kubernetes:main Jul 1, 2025
6 checks passed
neolit123 pushed a commit to neolit123/website that referenced this pull request Jul 2, 2025
…ubernetes#51357)

* [ja] modify content/ja/docs/contribute/localization.md

* Apply suggestions from code review

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* apply suggestion from code review, fix protocol schema of links

* Update content/ja/docs/contribute/localization.md

Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* Update content/ja/docs/contribute/localization.md

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>

---------

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
KobayashiD27 pushed a commit to KobayashiD27/website that referenced this pull request Jul 3, 2025
…ubernetes#51357)

* [ja] modify content/ja/docs/contribute/localization.md

* Apply suggestions from code review

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* apply suggestion from code review, fix protocol schema of links

* Update content/ja/docs/contribute/localization.md

Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* Update content/ja/docs/contribute/localization.md

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>

---------

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
iholder101 pushed a commit to iholder101/kubernetes-website that referenced this pull request Jul 3, 2025
…ubernetes#51357)

* [ja] modify content/ja/docs/contribute/localization.md

* Apply suggestions from code review

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* apply suggestion from code review, fix protocol schema of links

* Update content/ja/docs/contribute/localization.md

Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* Update content/ja/docs/contribute/localization.md

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>

---------

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
mochizuki875 pushed a commit to mochizuki875/website that referenced this pull request Jul 8, 2025
…ubernetes#51357)

* [ja] modify content/ja/docs/contribute/localization.md

* Apply suggestions from code review

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* apply suggestion from code review, fix protocol schema of links

* Update content/ja/docs/contribute/localization.md

Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>

* Update content/ja/docs/contribute/localization.md

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>

---------

Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/localization General issues or PRs related to localization cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/ja Issues or PRs related to Japanese language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ja] Clarify the localization process for kubernetes-specific terminologies (especially for boundary cases)
5 participants