各位专家,TC委员大家好:
为了提高
community 仓的
review 效率,减少错误发生,除了当前的
CI 检查之外,我准备收集一份例行的
check list。
当前识别的点包括?
review_body +=
"[ ] PR的标题是否清晰易懂 ? /
Is the Title of this PR self-explain ?\n"
review_body +=
"[ ] PR的内容描述是否详细具体 ?
/ Is the description of this PR detailed ?\n"
review_body +=
"[ ] PR和实际代码修改是否一致 ?
/ Does the PR match the code changes ?\n"
sigs = check_maintainer_changes()
if sigs:
review_body +=
"发现维护者变更 / Changes in
maintainship detected: \n"
review_body +=
"[ ]
如果新增维护者,有没有对他/她能力的客观说明
? / If new maintainer added, are there any assertion on his/her capability on maintainship ?\n"
for sig
in sigs:
review_body +=
"[ ] {sig}
中的其他维护者是否同意
增加/删除
维护者 ? / Do other maintainers of {sig} also agree to add/remove his/her as a maintainer ?\n".format(sig=sig)
if check_repository_mgmt_changes():
review_body +=
"发现 sigs.yaml
变更 / Changes in sigs.yaml detected: \n"
review_body +=
"[ ]
是否所有变更的代码仓都被恰当的 SIG
管理 ? / Are all changed repositories managed by proper SIG ?\n"
review_body +=
"[ ]
是否所有受影响的 SIG
的维护者都同意这个变更 ? / Do all affected SIG maintainers agree the change ?\n"
if check_repository_changes():
review_body +=
"[ ]
新引入的代码仓的license授权是否都与
openEuler 兼容 ? / Do newly introduced repository license compatible with openEuler ?\n"
请大家帮助审视反馈,还有哪些检查,在什么情况下应该被加到
openEuler community 的PR
审视中?
谢谢
Poincaré Laboratory
Huawei Technologies Co.,Ltd.