Pull Request已成功合入, 合并人@ascend-robot
(感谢 DaiHuina1 的贡献)变更摘要
本次 PR 将 Ascend/ascendc-kernelgen-data 仓库从 MindSDK/sigs/AgentSDK SIG 迁移至 common/sigs/agent-skills SIG,同时对该 SIG 的 sig-info.yaml 结构进行了重组,将原本顶层的 committers 字段下沉到各仓库条目内部。仓库的 committers 与 reviewers 人员配置保持不变。
主要改动
- 仓库归属迁移:
Ascend/ascendc-kernelgen-data仓库的配置信息(含 3 名 committers 和 5 名 reviewers)从MindSDK/sigs/AgentSDK/sig-info.yaml中移除,整体迁移至common/sigs/agent-skills/sig-info.yaml,归属于 agent-skills SIG。 - 文件结构重组:
common/sigs/agent-skills/sig-info.yaml中,原本定义在文件顶级的committers字段被移入Ascend/agent-skills仓库条目内部,使各仓库的committers成为仓库级配置,与Ascend/ascendc-kernelgen-data仓库的配置结构保持一致。 - 人员配置保留:迁移后的
ascendc-kernelgen-data仓库的 committers(冯彤rxtfeng、戴慧娜DaiHuina1、赵林林zhaolinlin)和 reviewers(冯彤rxtfeng、戴慧娜DaiHuina1、赵林林zhaolinlin、刘莹莹liuyy611、张琪家gcw_SUaZx3UQ)均原样保留,未做增删。


代码审查
审查总结
| 文件 | 审查结果 |
|---|---|
MindSDK/sigs/AgentSDK/sig-info.yaml |
✅ 无问题 — 仅删除了 ascendc-kernelgen-data 仓库条目,文件结构保持完整,顶层 committers: [] 和 reviewers: [] 均保留。 |
common/sigs/agent-skills/sig-info.yaml |
⚠️ 发现 1 个问题 (P2) — 缺失顶层 committers: 和 reviewers: 字段 |
问题统计:
- P0: 0
- P1: 0
- P2: 1
- P3: 0
整体风险评估:中等。 该变更在仓库迁移和权限数据迁移方面是正确的(ascendc-kernelgen-data 的 committer/reviewer 名单完整保留,agent-skills 原有 committer 名单也完整保留)。唯一的结构性问题是在 agent-skills 的 sig-info.yaml 中遗漏了顶层 committers: [] 和 reviewers: [] 占位字段,这与仓库中所有其他 sig-info.yaml 的格式约定不一致,可能导致 CI 校验失败或下游权限工具行为异常。建议在合入前补充这两个字段。
⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。


Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| sig-AgentSDK | ✅ 于金正, 杨宏伟 (2/2) | ✅ 于金正 (1/1) |
| sig-agent-skills | ✅ 许涛, 陈小刚 (2/2) | ✅ 许涛, 陈小刚 (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
DaiHuina1, thanks for your pull request. All authors of the commits have signed the CLA. 👍


ascend docs pipeline is running...


✅ 规则检查门禁通过!
| 检查项 | 检查结果 | 错误详情 |
|---|---|---|
| YAML格式检查 | ✅ 已通过 | - |
| 必须字段检查 | ✅ 已通过 | - |
| gitcode_id存在性检查 | ✅ 已通过 | - |
| 仓库存在性检查 | ✅ 已通过 | - |
| 文件位置检查 | ✅ 已通过 | - |
| 字段一致性检查 | ✅ 已通过 | - |
| maintainer数量检查 | ✅ 已通过 | - |
| 仓库类型检查 | ✅ 已通过 | - |
| sig名唯一性检查 | ✅ 已通过 | - |
| sig目录注册检查 | ✅ 已通过 | - |


✅ 跳过 docs ci 检查,没有需要检查的文档文件


ascend docs pipeline is running...


/lgtm


✅ 跳过 docs ci 检查,没有需要检查的文档文件


✅ 规则检查门禁通过!
| 检查项 | 检查结果 | 错误详情 |
|---|---|---|
| YAML格式检查 | ✅ 已通过 | - |
| 必须字段检查 | ✅ 已通过 | - |
| gitcode_id存在性检查 | ✅ 已通过 | - |
| 仓库存在性检查 | ✅ 已通过 | - |
| 文件位置检查 | ✅ 已通过 | - |
| 字段一致性检查 | ✅ 已通过 | - |
| maintainer数量检查 | ✅ 已通过 | - |
| 仓库类型检查 | ✅ 已通过 | - |
| sig名唯一性检查 | ✅ 已通过 | - |
| sig目录注册检查 | ✅ 已通过 | - |


/lgtm


/lgtm


/approve


/lgtm


/approve


/approve
/lgtm


migrate ascendc-kernelgen-data repository to agent-skills SIG