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

Translate Readme Run Zookeeper in Chinese. #369

Merged
merged 3 commits into from
May 29, 2022

Conversation

harryteng9527
Copy link
Collaborator

此PR的目的為將Run Zookeeper的Readme翻譯成中文版。

@harryteng9527 harryteng9527 requested a review from chia7712 May 27, 2022 05:14
Copy link
Contributor

@chia7712 chia7712 left a comment

Choose a reason for hiding this comment

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

@harryteng9527 感謝幫忙寫文件,希望能多補充一些,讓閱讀者可以有一個基本「認知」

@harryteng9527
Copy link
Collaborator Author

已新增一些有關Zookeeper、腳本的說明

Copy link
Contributor

@chia7712 chia7712 left a comment

Choose a reason for hiding this comment

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

@qoo332001 感謝更新

這個文件的文字要正式一點,下面有幾個建議請看一下

@harryteng9527
Copy link
Collaborator Author

謝謝學長的批閱,已將一些贅字刪除

Copy link
Contributor

@chia7712 chia7712 left a comment

Choose a reason for hiding this comment

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

@qoo332001 感謝文件,還不錯,就麻煩你其他文件也比照這個品質辦理,謝謝

再麻煩你合併

@harryteng9527 harryteng9527 merged commit 89ee2d0 into opensource4you:main May 29, 2022
@chia7712
Copy link
Contributor

@harryteng9527 不好意思,我剛剛才注意到那個檔名不太適合,用空白的話在使用上會造成一些困擾(例如在終端機操作),可否麻煩你再開一隻PR將檔案的調整一下,例如改成run_zookeeper.md

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.

2 participants