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
提一个PR到 community 吧,这样整个过程方便有记录。
From: George [mailto:caozhi1214@qq.com] Sent: Thursday, November 12, 2020 4:15 PM To: Huxinwei huxinwei@huawei.com; tc@openeuler.org Cc: dev dev@openeuler.org Subject: [Dev] 回复:Re: [Tc] 申请修改仓库配置文件格式
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分支通常就是非保护分支
[cid:image001.png@01D6B914.AF3BBAA0]
3. Created_from 应该叫fork_from
--create_from字段是按照gitee页面新建分支操作上的提示取的,希望和码云一致
[cid:image002.png@01D6B914.AF3BBAA0]
4. 这样的branch 会一直增加下去,怎么表示已经衰退或者停止的分支?什么时候删除分支? --我们考虑删除分支和删除仓库类似,属于高危操作,当前建议是不通过配置文件执行自动删除, 如果确实需要删除可以 将该分支从配置文件中删除,然后再到码云仓库中手动删除
5. 你们这个应该只考虑 src-openeuler.yaml 就可以了吧?我没想到需要改 openEuler.yaml 的场景。 -- 如果src-openeuler.yaml改了格式,openeuler.yaml格式不调整, CI-bot需要两套处理逻辑,增加了复杂度。
附格式样例: [cid:image003.png@01D6B914.AF3BBAA0]
------------------ 原始邮件 ------------------ 发件人: "Huxinwei" <huxinwei@huawei.commailto:huxinwei@huawei.com>; 发送时间: 2020年11月12日(星期四) 中午1:01 收件人: "George"<caozhi1214@qq.commailto:caozhi1214@qq.com>;"tc@openeuler.orgmailto:tc@openeuler.org"<tc@openeuler.orgmailto:tc@openeuler.org>; 抄送: "dev"<dev@openeuler.orgmailto: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.orgmailto:tc@openeuler.org Cc: dev <dev@openeuler.orgmailto:dev@openeuler.org> Subject: [Tc] 申请修改仓库配置文件格式
Hi,各位TC委员: 我是openEuler基础设施SIG的同事。 我们在落地【根据仓库配置文件自动创建仓库分支】的需求, 需要指定新建分支是基于哪个分支来创建; 当前community仓库下配置管理文件 src-openeuler.yaml和openeuler.yaml 的配置项没有包含这个信息,需要新增。 现申请修改两个仓库配置文件(src-openeuler.yaml和openeuler.yaml) 格式如下,烦请评审一下,谢谢。
老格式: [cid:image004.png@01D6B914.AF3BBAA0]
新格式: [cid:image005.png@01D6B914.AF3BBAA0] 注:create_from若为空,默认create_from到master, type若为空,默认protected