-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
Add Traditional Chinese translation for kubectl #46559
Conversation
Hi @warmchang. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
@zhangxiaoyu-zidif: changing LGTM is restricted to assignees, and only kubernetes org members may be assigned issues. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/assign @janetkuo |
|
||
#: pkg/kubectl/cmd/config/delete_cluster.go:39 | ||
msgid "Delete the specified cluster from the kubeconfig" | ||
msgstr "刪除 kubeconfig 檔中指定的集群" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
文檔 ? 文件 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, I can adjust the "檔" to "文檔" or "檔案", any other suggestions?
@shiywang , I've changed all "檔" to "文檔", rebuild the "bindata.go" & "k8s.mo". |
@k8s-bot ok to test |
|
||
#: pkg/kubectl/cmd/config/delete_cluster.go:39 | ||
msgid "Delete the specified cluster from the kubeconfig" | ||
msgstr "刪除 kubeconfig 文檔中指定的集群" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change all 文檔
to 檔案
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
集群
--> 叢集
, or 叢集(cluster)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, I will change them.
|
||
#: pkg/kubectl/cmd/config/delete_context.go:39 | ||
msgid "Delete the specified context from the kubeconfig" | ||
msgstr "刪除 kubeconfig 文檔中指定的上下文" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd suggest not translating context
, since it's a specific term used in kubeconfig
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, context
is more like 環境
, not 上下文
. You'd probably need to update the zh_CN one too.
You can also do 環境(context)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shiywang , you may consider this suggestion about the zh_CN translation of context.
I'll keep context not translating in zh_TW.
|
||
#: pkg/kubectl/cmd/config/set.go:60 | ||
msgid "Sets an individual value in a kubeconfig file" | ||
msgstr "設置 kubeconfig 文檔中的一個單個值" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
一個單個值
--> 一個值
or 某個值
|
||
#: pkg/kubectl/cmd/annotate.go:116 | ||
msgid "Update the annotations on a resource" | ||
msgstr "更新一個資源的注解" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
注解(annotation)
, since it's a special term in K8s
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you mean keep annotation not translating, just like the context?
msgid_plural "" | ||
"watch is only supported on individual resources and resource collections - " | ||
"%d resources were found" | ||
msgstr[0] "watch 僅支援單獨的資源或者資源集合 - 找到了 %d 個資源" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Edit: watch 僅支援單獨的資源或者資源集合
--> 一次只能 watch 一個資源或資料集合
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll change it, thanks!
@janetkuo , I've changed the translation, rebased the master branch and committed. |
|
||
#: pkg/kubectl/cmd/config/current_context.go:49 | ||
msgid "Displays the current-context" | ||
msgstr "顯示當前的context" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
當前
--> 目前
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some final comments, LGTM otherwise
|
||
#: pkg/kubectl/cmd/config/use_context.go:49 | ||
msgid "Sets the current-context in a kubeconfig file" | ||
msgstr "設置 kubeconfig 檔案中的當前context" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
當前
--> 目前
|
||
#: pkg/kubectl/cmd/annotate.go:116 | ||
msgid "Update the annotations on a resource" | ||
msgstr "更新一個資源的annotations" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be 注解(annotations)
or just annotations
, it's up to you
|
||
#: pkg/kubectl/cmd/config/delete_context.go:39 | ||
msgid "Delete the specified context from the kubeconfig" | ||
msgstr "刪除 kubeconfig 檔案中指定的context" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Optional: Add a space between Chinese & English character so that it looks consistent
@janetkuo , Based on your kind comments, I changed and committed it again. 😄 |
/release-note-none |
/assign @brendandburns |
@k8s-bot pull-kubernetes-federation-e2e-gce test this |
Resolve conflicts after #46756. |
@janetkuo @shashidharatd @brendandburns , all the test result are ok, please recheck this, and approve to merge, thx! |
@warmchang would you squash the commits into one? |
@janetkuo , thank you for your advice. I've done, and waiting for the end of the pull-kubernetes-xxx test. |
@warmchang: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/lgtm |
/approve no-issue Pre-code freeze LGTM, plus very low risk == approve. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brendandburns, janetkuo, warmchang Associated issue requirement bypassed by: brendandburns The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue |
Automatic merge from submit-queue (batch tested with PRs 46986, 51214, 51169, 50155, 51261) Add Italian translation for kubectl **What this PR does / why we need it:** This PR provides first attempt to translate kubectl in Italian (related to #40645, #45573, #45562, #40591, #46559). **Which issue this PR fixes** _(`optional, in fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)_: fixes # No issues **Special notes for your reviewer:** This PR requires Italian people to assist in the review. I'm native in Italian with BS in Computer Science. ```release-note Adding Italian translation for kubectl ```
Automatic merge from submit-queue Add German translation for kubectl **What this PR does / why we need it**: This PR provides a first attempt to translate kubectl in German (related to #40645, #45573, #45562, #40591, #46559, #50155). **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # No issues **Special notes for your reviewer**: This PR requires German people to assist in the review. I'm native in German with BSc in Business Information Technology. **Release note**: ```release-note Adding German translation for kubectl ```
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Adds initial Korean translations for kubectl **What this PR does / why we need it**: This PR provides a first attempt to translate kubectl in Korean (related to kubernetes#51867, kubernetes#40645, kubernetes#45573, kubernetes#45562, kubernetes#40591, kubernetes#46559, kubernetes#50155). **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: No issues **Special notes for your reviewer**: This PR requires Korean people to assist in the review. I am native in Korean and also a translator & reviewer in Korean in OpenStack I18n team (http://stackalytics.com/?metric=translations&user_id=ianychoi ). **Release note**: ```release-note Adding initial Korean translation for kubectl ```
What this PR does / why we need it:
This PR provides first attempt to translate kubectl in Traditional Chinese (related to #40645, #45573,
#45562, #40591).
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #No issues
Special notes for your reviewer:
This PR requires Chinese people to assist in the review, especially Chinese speaking Taiwanese or Cantonese, thanks!