summaryrefslogtreecommitdiff
path: root/.github
diff options
context:
space:
mode:
authorBernard Xiong <[email protected]>2019-01-02 13:11:13 +0800
committerBernard Xiong <[email protected]>2019-01-02 13:11:13 +0800
commitd9a3cc9a03e07c61afc1262a8e3a730cdc20d10b (patch)
tree5574009bbdb08bc4fbcfc6d669eaf6a1604e6e6d /.github
parentdc605b2c1f54bb631c10981a6dacef0ecc6d166a (diff)
Update PR template.
Diffstat (limited to '.github')
-rw-r--r--.github/PULL_REQUEST_TEMPLATE.md29
1 files changed, 16 insertions, 13 deletions
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index ffe25a7a43..c1e467e6c4 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -1,31 +1,34 @@
-### Summary of this Pull Request (PR) 拉取请求的简述
+### Summary of this Pull Request (PR) 拉取/合并请求的简述
-**Add description here.**
+**Add description here.** **请在这里加入描述**
-### Intent for your PR 拉取请求的目的
+### Intent for your PR 拉取/合并请求的目的
Choose one (Mandatory): 必须选择一项
-- [ ] This PR is for a code-review and is intended to get feedback 本拉取请求是一个草稿版本
-- [ ] This PR is mature, and ready to be integrated into the repo 本拉取请求是一个成熟版本
+- [ ] This PR is for a code-review and is intended to get feedback 本拉取/合并请求是一个草稿版本
+- [ ] This PR is mature, and ready to be integrated into the repo 本拉取/合并请求是一个成熟版本
### Reviewers (Mandatory): 代码审阅者(必须指定)
-(@<github.com username(s)> Ex: @user1, @user2)
+(@<github.com username(s)> Ex: @user1, @user2)
### Code Quality: 代码质量
+
As part of this pull request, I've considered the following:
-我在这个拉取请求中考虑了:
-- [ ] Style guide is adhered to, including spacing, naming and other style 代码的格式正确
-- [ ] All redundant code is removed and cleaned up 没有垃圾代码
-- [ ] All modifications are justified 所有变更均有原因
-- [ ] I've commented appropriately where code is tricky 难懂代码均被注释
-- [ ] Code in this PR is of high quality 本拉取请求高质量
+我在这个拉取/合并请求中已经考虑了:
+
+- [ ] Already check the difference between PR and old code 已经仔细查看过代码改动的对比
+- [ ] Style guide is adhered to, including spacing, naming and other style 代码风格正确,包括缩进空格,命名及其他风格
+- [ ] All redundant code is removed and cleaned up 没有垃圾代码,代码尽量精简,不包含`#if 0`代码,不包含已经被注释了的代码
+- [ ] All modifications are justified and not affect other components or BSP 所有变更均有原因及合理的,并且不会影响到其他软件组件代码或BSP
+- [ ] I've commented appropriately where code is tricky 对难懂代码均提供对应的注释
+- [ ] Code in this PR is of high quality 本拉取/合并请求代码是高质量的
### Testing:代码测试
I've tested the code using the following test programs (provide list here):
-我已经在如下用例上跑过测试:
+我已经在如下场合跑过对应的测试:
- [ ] application 1
- [ ] application 2