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

[zh-cn]sync api-extension/_index.md resource-quotas.md #46373

Merged
merged 1 commit into from
May 17, 2024

Conversation

my-git9
Copy link
Member

@my-git9 my-git9 commented May 14, 2024

  • Sync docs:
    • content/zh-cn/docs/concepts/extend-kubernetes/api-extension/_index.md
    • content/zh-cn/docs/concepts/policy/resource-quotas.md
  • Regarding the translation of namespaces in this article, there are more than 50 translations of "命名空间" and more than 10 translations of "名字空间". To avoid misunderstandings, they have been adjusted to "命名空间".

@k8s-ci-robot k8s-ci-robot added the area/localization General issues or PRs related to localization label May 14, 2024
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/zh Issues or PRs related to Chinese language size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels May 14, 2024
Copy link

netlify bot commented May 14, 2024

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 7d9eee6
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/6646b63786c14f0008598ea7
😎 Deploy Preview https://deploy-preview-46373--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 site configuration.

@tengqm
Copy link
Contributor

tengqm commented May 14, 2024

Regarding the translation of namespaces in this article, there are more than 50 translations of "命名空间" and more than 10 translations of "名字空间". To avoid misunderstandings, they have been adjusted to "命名空间".

We have had this discussion before, several times. We had agreed to keep both translations as is. We agreed to be neutral at this, which means no preference. The two translations are both acceptable. There is no confusion or misunderstanding. I don't think unifying this is needed. Instead, I'm worrying that we may provoke another round of unnecessary conflicts.

@my-git9
Copy link
Member Author

my-git9 commented May 15, 2024

We have had this discussion before, several times. We had agreed to keep both translations as is. We agreed to be neutral at this, which means no preference. The two translations are both acceptable. There is no confusion or misunderstanding. I don't think unifying this is needed. Instead, I'm worrying that we may provoke another round of unnecessary conflicts.

Yes, I'm aware of this situation. But wouldn’t it be better to guarantee one translation result for the same word in the same article?

@tengqm
Copy link
Contributor

tengqm commented May 15, 2024

We have had this discussion before, several times. We had agreed to keep both translations as is. We agreed to be neutral at this, which means no preference. The two translations are both acceptable. There is no confusion or misunderstanding. I don't think unifying this is needed. Instead, I'm worrying that we may provoke another round of unnecessary conflicts.

Yes, I'm aware of this situation. But wouldn’t it be better to guarantee one translation result for the same word in the same article?

For consistency's sake, using the same translation in one page is okay. What I'm concerned is that this kind of changes may trigger another initiative to do a site-wide change.

@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels May 16, 2024
@my-git9 my-git9 force-pushed the pt-22895 branch 3 times, most recently from 2a1c983 to b531ae2 Compare May 16, 2024 12:43
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 16, 2024
@my-git9
Copy link
Member Author

my-git9 commented May 16, 2024

We have had this discussion before, several times. We had agreed to keep both translations as is. We agreed to be neutral at this, which means no preference. The two translations are both acceptable. There is no confusion or misunderstanding. I don't think unifying this is needed. Instead, I'm worrying that we may provoke another round of unnecessary conflicts.

Yes, I'm aware of this situation. But wouldn’t it be better to guarantee one translation result for the same word in the same article?

For consistency's sake, using the same translation in one page is okay. What I'm concerned is that this kind of changes may trigger another initiative to do a site-wide change.

Modifications related to namespace have been restored

@tengqm
Copy link
Contributor

tengqm commented May 17, 2024

/lgtm

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

LGTM label has been added.

Git tree hash: c5aa00673ec18ce69ae19c012418e57e3e13bdcc

Copy link
Member

@ydFu ydFu left a comment

Choose a reason for hiding this comment

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

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ydFu

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 May 17, 2024
@k8s-ci-robot k8s-ci-robot merged commit 59c3eb3 into kubernetes:main May 17, 2024
6 checks passed
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/zh Issues or PRs related to Chinese language lgtm "Looks good to me", indicates that a PR is ready to be merged. 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.

None yet

4 participants