各位专家,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.