From 9887a2b9452af46aca2a809f192ad59d368de6a7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E6=9C=A8=E5=BE=97=E6=84=9F=E6=83=85=E7=9A=84openEuler?= =?UTF-8?q?=E6=9C=BA=E5=99=A8=E4=BA=BA?= Date: Tue, 8 Sep 2020 09:27:13 +0800 Subject: [PATCH] =?UTF-8?q?=E5=8F=98=E6=9B=B4review=E4=BF=A1=E6=81=AF?= =?UTF-8?q?=E6=A0=BC=E5=BC=8F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- advisors/tc_review | 23 ++++++++++++----------- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/advisors/tc_review b/advisors/tc_review index 03eba151..6f845b8f 100755 --- a/advisors/tc_review +++ b/advisors/tc_review @@ -48,22 +48,23 @@ def check_maintainer_changes(): def review(pr): - review_body = "" - 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" + review_body = "**以下为 openEuler-Advisor 的 tc_review 生成审视要求清单**\n" + review_body += "[Y] 审视者确认符合要求 [N] 审视者认为不符合要求 [?] 审视者无法确认是否符合要求 [] 审视过程中\n" + review_body += "|审视情况|审视要求|审视要求说明|\n" + review_body += "|:--:|:--|:--|\n" + review_body += "|[ ]|PR的标题是否清晰易懂?|提交标题应该一句话说明本提交实现的内容。|\n" + review_body += "|[ ]|PR的内容描述是否详细具体?|提交的描述应该用一段话说明本提交的背景和实现原理。|\n" + review_body += "|[ ]|PR和实际代码修改是否一致?|提交的说明文字应该和实际代码修改内容保持一致。|\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" + review_body += "|[ ]|如果新增维护者，有没有对他/她能力的客观说明?|PR提交者需要提出相应的举证说明维护者候选人的技术能力与社区活跃程度。|\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) + review_body += "|[ ]|{sig} 中的其他维护者是否同意增加/删除维护者?|{sig}当前维护者需要有代表确认是否同意接纳或者移除维护者。|\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" + review_body += "|[ ]|是否所有变更的代码仓都被恰当的 SIG 管理?|代码仓应当由有能力且有意愿的SIG管理，同一类的软件尽量归属同一个SIG。|\n" + review_body += "|[ ]|是否所有受影响的 SIG 的维护者都同意这个变更?|每一个涉及的 SIG 都需要有维护者代表确认是否同意变更。|\n" if check_repository_changes(): - review_body += "[ ] 新引入的代码仓的license授权是否都与 openEuler 兼容 ? / Do newly introduced repository license compatible with openEuler ?\n" + review_body += "|[ ]|新引入的代码仓的 license 授权是否都与 openEuler 兼容?|openEuler只能接纳可以被允许集成的软件|\n" return review_body def main(): -- GitLab