-
-
Notifications
You must be signed in to change notification settings - Fork 214
Translate CSV Module rst:71 to rst:152 #582
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
Conversation
啊....是不是因為我共用前幾天的 PR 的 Branch 造成多個 commit .....@@ |
對歐,可以學一下 |
@cschan1828 想請問最近有空來幫忙 review 嗎? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the commit!
有些常見的動詞其實可以考慮直接找中文慣用動詞來用哦,保留原文主要是指專有名詞。
Hi @cschan1828 , thanks for the review! 感謝你的細心,再麻煩幫我看看了,感謝! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
相當不錯了,僅想再釐清一些小細節。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
很通用的詞語直接用中文即可。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Approved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks guys!
* Translate rst:71 to rst:152 * Update with Reviewer's comment * HOTFIX
#188