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