Pull Request已成功合入, 合并人@CANN-robot
（感谢 邢智雄 的贡献）
代码审查
审查结束总结
| 文件 | 审查结果 |
|---|---|
docs/en/precommit_guide.md |
发现 2 个问题（1×P1, 1×P3） |
docs/zh/precommit_guide.md |
发现 2 个问题（1×P1, 1×P3） |
按优先级统计：
- P1：2 个（英文版和中文版各一个，均为同一问题——文档教导用户使用
--no-verify绕过合规检查） - P3：2 个（英文版和中文版各一个，均为同一问题——
curl \| bash无安全警告）
整体风险评估：中等风险。 核心问题在于文档在"功能概述"中明确表示合规阻止"并非强制、可以忽略"，并在后续章节中直接教导 git commit --no-verify 绕过方法，这与 pre-commit 的合规管控目标直接矛盾，可能导致不合规代码（二进制文件、缺少许可证头）被提交到仓库。建议优先修改这两处表述。
| 类型 | 数量 |
|---|---|
| 🔴 阻塞 | 2 |
| 🟡 建议 | 0 |
⛔ 需要修改
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 | ✅ yangyongqiang0606, xuyafei, wangxiaotian995 (3/2) | ✅ yangyongqiang0606 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
xingzhixiong, thanks for your pull request. All authors of the commits have signed the CLA. 👍
CANN-robotCANN-robot
compile
邢智雄流水线任务触发成功
任务链接 [0a037c7889634e79bffbae431772cb74][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| UT_Test_Python_superkernel | ✅ SUCCESS | >>>>> | |
| ST_Test_Python_superkernel | ✅ SUCCESS | >>>>> | |
| UT_Test_superkernel | ✅ SUCCESS | >>>>> | |
| UT_Test_autofuse_framework | ✅ SUCCESS | >>>>> | |
| ST_Test_autofuse_framework | ✅ SUCCESS | >>>>> | |
| UT_Test_autofuse_ascendc_api | ✅ SUCCESS | >>>>> | |
| ST_Test_autofuse_ascendc_api | ✅ SUCCESS | >>>>> | |
| ST_Test_autofuse_e2e | ✅ SUCCESS | >>>>> | |
| pre_comment | ✅ SUCCESS | >>>>> |
[2026-07-15 16:37:42] CI执行结束
/lgtm
🟠 High Priority
英文版第 14 行明确声明 "Blocking is not forced - you can ignore the modifications."，且第 160-161 行在合规问题章节中直接将 git commit --no-verify 作为"temporarily skip this check"的解决方案展示给用户。
--no-verify 会绕过所有 pre-commit hooks（不仅仅是 OAT），包括代码格式化等。pre-commit 机制的设计初衷是在提交前强制进行合规检查，而文档却在教导用户如何跳过，这直接削弱了合规管控的有效性。用户在了解此方法后，可能在任何合规问题（二进制文件、许可证头缺失）出现时直接绕过，导致不合规代码进入仓库。
changed line → affected behavior: 第 14 行和第 160-161 行 → 用户获知可无条件绕过 pre-commit 检查 → 不合规代码可能被提交。
建议：删除或修改第 14 行的"可以忽略"表述，并将第 160-161 行的 --no-verify 绕过方法移除，或至少添加明确的警告说明这是最后的应急手段，不应在日常开发中常规使用，且使用后必须随后运行 pre-commit run oat-check 补检。
🟠 High Priority
中文版第 14 行明确声明 "阻止并非强制修改，可以忽略修改"，且第 160-161 行在合规问题章节中直接将 git commit --no-verify 作为"temporarily skip this check"的解决方案展示给用户。
--no-verify 会绕过所有 pre-commit hooks（不仅仅是 OAT），包括代码格式化等。pre-commit 机制的设计初衷是在提交前强制进行合规检查，而文档却在教导用户如何跳过，这直接削弱了合规管控的有效性。用户在了解此方法后，可能在任何合规问题（二进制文件、许可证头缺失）出现时直接绕过，导致不合规代码进入仓库。
changed line → affected behavior: 第 14 行和第 160-161 行 → 用户获知可无条件绕过 pre-commit 检查 → 不合规代码可能被提交。
建议：删除或修改第 14 行的"可以忽略修改"表述，并将第 160-161 行的 --no-verify 绕过方法移除，或至少添加明确的警告说明这是最后的应急手段，不应在日常开发中常规使用，且使用后必须随后运行 pre-commit run oat-check 补检。
/approve
Pull Request
描述
新增pre-commit指导文档。
变更类型
请选择本次引入的变更类型：
关联的Issue
如何测试
描述测试此变更的步骤和前提条件：
核对清单
其他信息
在此添加任何其他关于本次 PR 的说明。