Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
openeuler
openEuler-Advisor
提交
9887a2b9
O
openEuler-Advisor
项目概览
openeuler
/
openEuler-Advisor
通知
40
Star
4
Fork
4
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
O
openEuler-Advisor
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
提交
9887a2b9
编写于
9月 08, 2020
作者:
木
木得感情的openEuler机器人
提交者:
Gitee
9月 08, 2020
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
变更review信息格式
上级
5365f21b
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
12 addition
and
11 deletion
+12
-11
advisors/tc_review
advisors/tc_review
+12
-11
未找到文件。
advisors/tc_review
浏览文件 @
9887a2b9
...
...
@@ -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
():
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录