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

[FLINK-11567] Translate "How to Review a Pull Request" page into Chinese #176

Closed
wants to merge 1 commit into from

Conversation

linjie98
Copy link
Contributor

Translate "How to Review a Pull Request" page into Chinese.

The markdown file is located in: flink-web/reviewing-prs.zh.md
The url link is: https://flink.apache.org/zh/reviewing-prs.html

Notes: This PR has fixed all the problems from "#171". Because of my improper operation, the PR of "#171" can not be changed any more. Thank you for your understanding.

Copy link
Contributor

@packbase packbase left a comment

Choose a reason for hiding this comment

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

@ash-ali Thanks for your work. It look good to me. I have left some minor comments.

Best
Kaibo


**A contribution is well-described if the following questions 2, 3, and 4 can be answered without looking at the code.**
**如果在不查看代码的情况下能回答以下问题2、3、4,则该贡献得到了很好的描述。**
Copy link
Contributor

Choose a reason for hiding this comment

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

以下问题2、3、4
->
以下问题2、3和4

Copy link
Member

Choose a reason for hiding this comment

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

和字加上有什么意义?


**A feature, improvement, or bug fix is approved once one committer accepts it and no committer disagrees (lazy consensus).**
**一旦有一个 committer 接受了某个功能、改进或者错误修复,并且没有其他 committer 不同意(lazy consensus 机制),那么就会批准它。**
Copy link
Contributor

Choose a reason for hiding this comment

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

那么就会批准它。
->
那么该特性、改进或错误修复就会得到批准。

@linjie98 linjie98 closed this Feb 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants