已合并
chore: update classify unrelease rules #1535
chore: update classify unrelease rules #1535
已合并
ling-DT创建于 7月28日
ling-DT成员
7月28日

Pull Request

描述

autofuse 加入 classify_rule.yamlrelease 分类,并将 autofuse/graph_metadefautofuse/examples 加入 unrelease 分类。

变更类型

关联的Issue

无。

如何测试

描述测试此变更的步骤和前提条件:

  1. python3 -c "import yaml; yaml.safe_load(open('classify_rule.yaml', encoding='utf-8'))"
  2. git diff --check

核对清单

其他信息

本 PR 只包含 classify_rule.yaml 三行分类规则新增。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 ling-DT 的贡献)
Lling-DT成员
7月28日 创建了 pull request,commit 00e22983
atomgit-bot
atomgit-bot
7月28日 评论:

变更摘要

本次变更更新了 classify_rule.yaml 分类规则配置文件,主要调整了 superkernel 模块下 autofuse 相关路径的发布/未发布分类归属,将 autofuse 主体源码归入 release 类别,同时将 autofuse 下的 graph_metadefexamples 目录归入 unrelease 类别。

主要改动

  • autofuse 整体纳入 release 分类:在 superkernel.src.release 下新增 autofuse 路径,将 autofuse 主体源码标记为可发布内容。
  • autofuse/graph_metadef 归入 unrelease 分类:在 superkernel.src.unrelease 下新增 autofuse/graph_metadef,将图元定义相关目录排除在发布范围之外。
  • autofuse/examples 归入 unrelease 分类:在 superkernel.src.unrelease 下新增 autofuse/examples,将示例代码目录排除在发布范围之外。
likedislike
atomgit-bot
atomgit-bot
7月28日 评论:

代码审查

审查总结

classify_rule.yaml 的变更进行了审查。

审查结果

文件 结果
classify_rule.yaml 1 个 P2 发现

发现统计

  • P0: 0
  • P1: 0
  • P2: 1
  • P3: 0

总体风险判断

中等偏低。本次变更在 autofuse 的 release/unrelease 分类上采用了与既有 super_kernel 不同的路径重叠策略(release 全量目录 + unrelease 排除子目录 vs 互斥路径)。该重叠行为是否正确取决于分类系统的路径解析实现——若系统支持"更具体路径优先"则安全,否则可能导致 autofuse/graph_metadef 等子目录被误分类。建议在合并前确认分类系统对重叠路径的处理策略。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
CANN-robotCANN-robot成员
7月28日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7月28日 评论:

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
repo-cann/graph-autofusion zhanj, zhang_shengjie, xchu42 (3/2) zhanj (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

ling-DT, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
此处折叠了160条消息 查看更多
zhang_shengjie成员
7月29日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
7月29日 添加了label:lgtm
zhanj
zhanj成员
7月29日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
7月29日 添加了label:approved
CANN-robotCANN-robot成员
7月29日 合入了pull request