已关闭
【pre-commit】全仓.h/.hpp历史格式问题清理(为Regbase替换扫清钩子障碍) #4850
景明创建于  9月1日关闭于  9月2日
景明
景明成员
9月1日 创建

问题描述

仓内大量 .h/.hpp 文件存在历史格式问题(尾随空白、文件末尾缺换行、clang-format 不合规),导致任何后续功能改动 PR 一旦触发 pre-commit 就会被大面积无关的格式化 diff 污染,无法聚焦 review 真实改动。需要先全仓清理,为 Regbase(MicroAPI→Reg)替换等后续工作扫清钩子障碍。

清理方案

find <目录> -type f \( -name '*.h' -o -name '*.hpp' \) | xargs pre-commit run --files
# 多轮执行直到所有 hook 收敛

排除范围

  • *_def.cpp 算子定义文件(见 Issue #4848 / PR #11041)
  • */op_api/aclnn*.h、*/op_graph/*_proto.h、*/op_host/op_api/aclnn*.h 接口/原型头文件(排版由开发者自行维护)
  • mamba/ 目录(causal_conv1d UT 上游基线失败,见 Issue #4840)
  • examples/、experimental/ 目录

拆分 PR(共 2362 个文件,6 个 PR 互不重叠)

PR 分支 范围
#4878 compile attention 第1批
#10422 graph attention 第2批
#9331 test attention 第3批
#10991 fix mc2
#11020 fix_2 moe、gmm
#11021 fix_3 posembedding、mhc、ffn、common、tests、torch_extension

验证

  • 纯格式化改动,不改变任何代码语义
  • 各分支 pre-commit 全量 hook 收敛(trailing-whitespace / EOF / clang-format 均 Passed)
likedislike
weihao18成员
9月1日 评论:

/assign @gitcode_lijd

likedislike
CANN-robotCANN-robot成员
9月1日 将 gitcode_lijd 设为负责人
景明
景明成员
9月2日 评论:

5/6 个关联 PR 已合入(#4878、#9331、#10991、#11020、#11021),#10422 因 CI 耗时问题另行跟踪,本 issue 关闭。

likedislike
景明景明成员
9月2日 修改标题为 “t”,原标题为“【pre-commit】全仓.h/.hpp历史格式问题清理(为Regbase替换扫清钩子障碍)”
景明景明成员
9月2日 修改标题为 “test”,原标题为“t”
景明景明成员
9月2日 修改标题为 “【pre-commit】全仓.h/.hpp历史格式问题清理(为Regbase替换扫清钩子障碍)”,原标题为“test”
CANN-robotCANN-robot成员
9月2日 关闭了 issue
CANN-robotCANN-robot成员
9月2日 添加了label:resolved