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

Translation for dockerDeploy.md #595

Merged
merged 4 commits into from
Sep 2, 2022
Merged

Translation for dockerDeploy.md #595

merged 4 commits into from
Sep 2, 2022

Conversation

baozidai
Copy link
Contributor

@baozidai baozidai commented Sep 1, 2022

这份PR提交了docker快速部署的英文翻译意译

CONTRIBUTING.md说我应该向dev分支提交PR,但是我无法确定dev/change-doc-home-actions分支是否就是这个所谓的“dev”分支,在这里的说明,图片中提示向main分支提交PR,我也不确定我应该向哪个分支提交PR。如果我选择了错误的分支,请海涵由此带来的不便。

@cyw3 cyw3 requested a review from aaaa-j September 1, 2022 10:45
doc/en/README.md Outdated Show resolved Hide resolved
doc/en/quickStarted/dockerDeploy.md Outdated Show resolved Hide resolved
@aaaa-j aaaa-j requested a review from cyw3 September 2, 2022 02:17
修改关于“分布式客户端”的翻译
@baozidai
Copy link
Contributor Author

baozidai commented Sep 2, 2022

我无法确定是否有其他的地方通过路径引用了这些文件名,所以没有修改文件名

@cyw3
Copy link
Collaborator

cyw3 commented Sep 2, 2022

我无法确定是否有其他的地方通过路径引用了这些文件名,所以没有修改文件名

明白,这个后续再更新吧。赞

Copy link
Collaborator

@cyw3 cyw3 left a comment

Choose a reason for hiding this comment

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

LGTM

@aaaa-j
Copy link
Collaborator

aaaa-j commented Sep 2, 2022

LGTM

@cyw3 cyw3 merged commit 32d4605 into Tencent:main Sep 2, 2022
@baozidai baozidai mentioned this pull request Sep 2, 2022
@baozidai baozidai deleted the dev/translation_for_docker_deploy branch September 2, 2022 12:38
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