0. 从评审角度,以举例的方式来评审格式是不严谨的。希望有一个规范的格式文件,比如字段的属性,可选项等等。 -- 输出了 仓库配置管理格式说明.md 文件,看是否满足要求( https://github.com/GeorgeCao-hw/georgedoc/blob/master/yamlformat.md%EF%BC%89
1. 这个yaml文件格式看起来将来还有可能变化,那这次修改应该把格式版本字段加上。
-- 赞同该建议, 在yaml文件开头增加version字段
2. Branch的type normal 有什么意义吗?
-- branch的type是按照码云分支的类型设置,包括protected,normal,readyonly三个选项,normal分支通常就是非保护分支
3. Created_from 应该叫fork_from
--create_from字段是按照gitee页面新建分支操作上的提示取的,希望和码云一致
4. 这样的branch 会一直增加下去,怎么表示已经衰退或者停止的分支?什么时候删除分支?
--我们考虑删除分支和删除仓库类似,属于高危操作,当前建议是不通过配置文件执行自动删除, 如果确实需要删除可以 将该分支从配置文件中删除,然后再到码云仓库中手动删除
5. 你们这个应该只考虑 src-openeuler.yaml 就可以了吧?我没想到需要改 openEuler.yaml 的场景。 -- 如果src-openeuler.yaml改了格式,openeuler.yaml格式不调整, CI-bot需要两套处理逻辑,增加了复杂度。
附格式样例:
------------------ 原始邮件 ------------------ 发件人: "Huxinwei" <huxinwei@huawei.com>; 发送时间: 2020年11月12日(星期四) 中午1:01 收件人: "George"<caozhi1214@qq.com>;"tc@openeuler.org"<tc@openeuler.org>; 抄送: "dev"<dev@openeuler.org>; 主题: [Dev] Re: [Tc] 申请修改仓库配置文件格式
你们这个应该只考虑 src-openeuler.yaml 就可以了吧?我没想到需要改 openEuler.yaml 的场景。
以下是我的建议:
0. 从评审角度,以举例的方式来评审格式是不严谨的。希望有一个规范的格式文件,比如字段的属性,可选项等等。
1. 这个yaml文件格式看起来将来还有可能变化,那这次修改应该把格式版本字段加上。
2. Branch的type normal 有什么意义吗?
3. Created_from 应该叫fork_from
4. 这样的branch 会一直增加下去,怎么表示已经衰退或者停止的分支?什么时候删除分支?
From: George [mailto:caozhi1214@qq.com] Sent: Thursday, November 12, 2020 12:05 PM To: tc@openeuler.org Cc: dev <dev@openeuler.org> Subject: [Tc] 申请修改仓库配置文件格式
Hi,各位TC委员:
我是openEuler基础设施SIG的同事。
我们在落地【根据仓库配置文件自动创建仓库分支】的需求, 需要指定新建分支是基于哪个分支来创建;
当前community仓库下配置管理文件 src-openeuler.yaml和openeuler.yaml 的配置项没有包含这个信息,需要新增。
现申请修改两个仓库配置文件(src-openeuler.yaml和openeuler.yaml) 格式如下,烦请评审一下,谢谢。
老格式:
新格式:
注:create_from若为空,默认create_from到master,
type若为空,默认protected