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

Add Traditional Chinese translation for kubectl #46559

Merged
merged 1 commit into from
Jun 8, 2017
Merged

Add Traditional Chinese translation for kubectl #46559

merged 1 commit into from
Jun 8, 2017

Conversation

warmchang
Copy link
Contributor

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!

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 27, 2017
@k8s-ci-robot
Copy link
Contributor

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 @k8s-bot ok to test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

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.

@k8s-ci-robot k8s-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label May 27, 2017
@k8s-github-robot k8s-github-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. release-note-label-needed labels May 27, 2017
@k8s-ci-robot
Copy link
Contributor

@zhangxiaoyu-zidif: changing LGTM is restricted to assignees, and only kubernetes org members may be assigned issues.

In response to this:

/lgtm

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
Copy link
Contributor

/assign @janetkuo
You may have a good knowledge of Traditional Chinese.


#: pkg/kubectl/cmd/config/delete_cluster.go:39
msgid "Delete the specified cluster from the kubeconfig"
msgstr "刪除 kubeconfig 檔中指定的集群"
Copy link
Contributor

Choose a reason for hiding this comment

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

文檔 ? 文件 ?

Copy link
Contributor Author

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?

@deads2k deads2k removed their assignment May 30, 2017
@warmchang
Copy link
Contributor Author

@shiywang , I've changed all "檔" to "文檔", rebuild the "bindata.go" & "k8s.mo".
Pls check it and review again.

@janetkuo
Copy link
Member

@k8s-bot ok to test

@k8s-ci-robot k8s-ci-robot removed the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label May 31, 2017

#: pkg/kubectl/cmd/config/delete_cluster.go:39
msgid "Delete the specified cluster from the kubeconfig"
msgstr "刪除 kubeconfig 文檔中指定的集群"
Copy link
Member

Choose a reason for hiding this comment

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

change all 文檔 to 檔案

Copy link
Member

Choose a reason for hiding this comment

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

集群 --> 叢集, or 叢集(cluster)

Copy link
Contributor Author

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 文檔中指定的上下文"
Copy link
Member

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

Copy link
Member

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)

Copy link
Contributor Author

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 文檔中的一個單個值"
Copy link
Member

@janetkuo janetkuo May 31, 2017

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 "更新一個資源的注解"
Copy link
Member

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

Copy link
Contributor Author

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 個資源"
Copy link
Member

@janetkuo janetkuo May 31, 2017

Choose a reason for hiding this comment

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

Edit: watch 僅支援單獨的資源或者資源集合 --> 一次只能 watch 一個資源或資料集合

Copy link
Contributor Author

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!

@warmchang
Copy link
Contributor Author

@janetkuo , I've changed the translation, rebased the master branch and committed.
Please check it, thank you.


#: pkg/kubectl/cmd/config/current_context.go:49
msgid "Displays the current-context"
msgstr "顯示當前的context"
Copy link
Member

@janetkuo janetkuo Jun 1, 2017

Choose a reason for hiding this comment

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

當前 --> 目前

Copy link
Member

@janetkuo janetkuo left a 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"
Copy link
Member

@janetkuo janetkuo Jun 1, 2017

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"
Copy link
Member

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"
Copy link
Member

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

@warmchang
Copy link
Contributor Author

@janetkuo , Based on your kind comments, I changed and committed it again. 😄

@shashidharatd
Copy link

@k8s-bot pull-kubernetes-federation-e2e-gce test this
ref: #46827

@janetkuo
Copy link
Member

janetkuo commented Jun 2, 2017

/release-note-none
/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. and removed release-note-label-needed labels Jun 2, 2017
@warmchang
Copy link
Contributor Author

/assign @brendandburns
This PR has been approved by @janetkuo , Pls check it and make the additional decisions, thx.

@brendandburns
Copy link
Contributor

@k8s-bot pull-kubernetes-federation-e2e-gce test this

@brendandburns brendandburns added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Jun 7, 2017
@brendandburns brendandburns removed the release-note-none Denotes a PR that doesn't merit a release note. label Jun 7, 2017
@brendandburns brendandburns added this to the v1.7 milestone Jun 7, 2017
@k8s-github-robot k8s-github-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 7, 2017
@warmchang
Copy link
Contributor Author

Resolve conflicts after #46756.

@warmchang
Copy link
Contributor Author

@janetkuo @shashidharatd @brendandburns , all the test result are ok, please recheck this, and approve to merge, thx!

@janetkuo
Copy link
Member

janetkuo commented Jun 8, 2017

@warmchang would you squash the commits into one?

@warmchang
Copy link
Contributor Author

@janetkuo , thank you for your advice. I've done, and waiting for the end of the pull-kubernetes-xxx test.

@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Jun 8, 2017

@warmchang: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
pull-kubernetes-federation-e2e-gce dd2b7434f282208c260f4cd30ed621ee566cf4b5 link @k8s-bot pull-kubernetes-federation-e2e-gce test this

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.

@warmchang
Copy link
Contributor Author

warmchang commented Jun 8, 2017

@k8s-bot pull-kubernetes-e2e-gce-etcd3 test this
@k8s-bot pull-kubernetes-federation-e2e-gce test this

@brendandburns
Copy link
Contributor

/lgtm
/approve

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

/approve no-issue

Pre-code freeze LGTM, plus very low risk == approve.

@k8s-github-robot
Copy link

[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 /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-github-robot k8s-github-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 8, 2017
@marun marun added the sig/cli Categorizes an issue or PR as relevant to SIG CLI. label Jun 8, 2017
@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 165a17b into kubernetes:master Jun 8, 2017
k8s-github-robot pushed a commit that referenced this pull request Aug 25, 2017
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
```
k8s-github-robot pushed a commit that referenced this pull request Sep 8, 2017
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
```
feiskyer pushed a commit to feiskyer/kubernetes that referenced this pull request May 14, 2018
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
```
@warmchang warmchang deleted the zh_TW branch September 12, 2019 07:53
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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/cli Categorizes an issue or PR as relevant to SIG CLI. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.