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

翻译4.5.4 Webhook章节 #182

Merged
merged 2 commits into from
Aug 16, 2020
Merged

翻译4.5.4 Webhook章节 #182

merged 2 commits into from
Aug 16, 2020

Conversation

skyshang1
Copy link

ref: https://github.com/cloudnativeto/kubebuilder/issues/

[ ] Configuration Infrastructure
[x] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@netlify
Copy link

netlify bot commented Aug 12, 2020

Deploy succeeded

Built with commit 7c9c123

https://deploy-preview-182--cnc-kubebuilder.netlify.app

@skyshang1
Copy link
Author

/reference/markers/webhook.md #93

@hzliangbin
Copy link

/review

Use these to keep the description of your webhooks close to the code that
implements them.
这些标记描述了[webhook配置](../webhook-overview.md)如何生成。
使用这些使您的 webhook 描述与实现它们的代码保持一致。

{{#markerdocs Webhook}}

Choose a reason for hiding this comment

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

注意下看下预览界面,引用的webhook-overview.md部分是否需要翻译?

Copy link
Author

Choose a reason for hiding this comment

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

这个我就不知道了😅

Choose a reason for hiding this comment

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

这个不需要翻译,因为是单独的一章节,后面有单独的issue来翻译那个。

Copy link

@majinghe majinghe left a comment

Choose a reason for hiding this comment

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

@skyshang1 你看一下这些review意见,修改一些。多谢了

Use these to keep the description of your webhooks close to the code that
implements them.
这些标记描述了[webhook配置](../webhook-overview.md)如何生成。
使用这些使您的 webhook 描述与实现它们的代码保持一致。

Choose a reason for hiding this comment

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

把您改成你,咱们统一一下。

Copy link
Author

Choose a reason for hiding this comment

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

好的。

@skyshang1
Copy link
Author

@lhb008 这个修改完了,你看下能 merge 了不

@majinghe
Copy link

/LGTM

@mergify mergify bot merged commit c4d39f8 into cloudnativeto:zh Aug 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants