已关闭
同步 ops-cv pre-commit 配置 #635
zhaowenrui创建于  7月22日关闭于  7月24日
zhaowenrui
zhaowenrui成员
7月22日 创建

问题描述

当前 ops-cv 的 .pre-commit-config.yaml 与 ops-nn 仓配置不一致,缺少基础 pre-commit hooks、Python 检查和拼写检查等配置。需要按 ops-nn 补齐提交前检查能力,同时保持 Markdown 文件不参与 codespell 拼写检查,避免文档和代码片段触发拼写误报。

修改方案

  • 补齐基础 pre-commit hooks:trailing-whitespace、end-of-file-fixer、check-yaml、check-added-large-files、check-merge-conflict、detect-private-key、check-json。
  • 对齐 clang-format 配置:使用 v18.1.8,增加 --style=file,覆盖 c/h/cpp/hpp/cc/hh/cxx/hxx/asc。
  • 增加 Python 检查:ruff-check 和 ruff-format。
  • 增加拼写检查:codespell v2.4.1。
  • 对齐 OAT hook 的 stages: [pre-commit]。
  • codespell skip 列表保留 *.md,对齐 ops-math 中 Markdown 不检查拼写的处理。

影响范围

本次仅修改 pre-commit 配置,不涉及算子实现、编译脚本和运行逻辑。影响范围集中在后续提交时的本地/流水线提交前检查。

关联 PR

验证结果

  • pre-commit validate-config:通过。
  • pre-commit run --files .pre-commit-config.yaml:通过。
    • basic hooks:通过。
    • codespell:通过。
    • OAT:通过。
likedislike
zhaowenruizhaowenrui成员
7月22日 修改标题为 “同步 ops-cv pre-commit 配置”,原标题为“pre-commit 配置同步 ops-nn 并验证通过”
zhaowenruizhaowenrui成员
7月22日 修改了issue 的描述
Mmajiajian_hw成员
7月22日 将 zhaowenrui666 设为负责人
zhaowenruizhaowenrui成员
7月22日 关联了pull request:chore: sync pre-commit config with ops-nn
CANN-robotCANN-robot成员
7月24日 关闭了 issue
CANN-robotCANN-robot成员
7月24日 添加了label:resolved