已合并
feat(governance): 注册 catlass-dsl 路由至 ops 域(三仓迁移 1/4) #126
feat(governance): 注册 catlass-dsl 路由至 ops 域(三仓迁移 1/4) #126
已合并
longjihui创建于 11 天前
longjihui
longjihui
11 天前

Pull Request 说明

变更说明

  • 关联 Issue:无
  • 主要改动:为接收 cannbot-skills 仓 catlass-dsl-generator 插件迁移产出的 CATLASS Python DSL 知识,注册规则 route:governance/schemas/registries.yaml 的 technologies 新增 catlass-dsl(type: dsl、domains: [ops],CATLASS Python 算子编程 DSL),与 !122 已合入的 catlass(type: ascendc,C++ 模板栈)互补共存:前者是 Python DSL,后者是 C++ 技术栈。
  • 配套:governance/contracts/knowledge.py 的 ID_RE 放宽为 ^[a-z0-9_-]+$(允许连字符技术 ID,卡片文件名/section 约束不受影响);governance/specs/paths.md ops route 表补该 route(文档一致性测试要求);evals/test_governance.py 预建 route 期望集合补 knowledge/ops/catlass-dsl(对齐 !122 的最小改动形态)。
  • 占位收口 !122 过渡红窗(评审要求):!122 注册 catlass 后其内容 PR 未合入期间,基线快照报 structure.missing: knowledge/ops/catlass 阻塞其他 PR 的 FormatCheck。本 PR 补 knowledge/ops/catlass/index.md 占位入口与 knowledge/ops/index.md 导航行——catlass 规则已在基线、knowledge/ 进快照,快照口径实测 P0=0;johan_lee 的内容 PR 合入时真 index 直接覆盖占位即可。
  • catlass-dsl 自身的 route 目录不随本 PR:其注册规则在本 PR 内、不进 trusted snapshot,规则 PR 携带自身 index 会被判 unregistered P0(!122 作者实测结论)。catlass-dsl 目录与导航行随内容 PR !127 紧随合入,合入前基线存在 structure.missing: knowledge/ops/catlass-dsl 的过渡窗口,与 !122 模式一致。

变更类型

验证

说明:本地全树口径 test_registered_structure_is_materialized / test_current_index_tree_is_complete 因 knowledge/ops/catlass-dsl 目录未随本 PR 落地暂红(预期过渡态,!127 合入后恢复,与 !122 合入时相同);CI FormatCheck 走快照口径不受影响。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 longjihui 的贡献)
longjihuilongjihui
11 天前 创建了 pull request,commit 80034b0d
CANN-robot
CANN-robot成员
11 天前 评论:

Hi @longjihui, welcome to submitting your first PR to cannbot-knowledge!

PR Merge Steps

1. CLA Signing

If the current PR label includes cann-cla/yes, it means you have signed the CLA and can proceed to the next step. If the label includes cann-cla/no, please sign the CLA first. If you have any questions, please refer to the FAQ.

2. CI Check

Please comment /compile to trigger the CI pipeline check. If the CI run is successful, the PR will be tagged with ci-pipeline-passed and you can proceed to the next step. If the CI run fails, the PR will be tagged with ci-pipeline-failed, please check the CI logs to fix the issues in the PR. If you have any questions, please refer to the FAQ.

3. Code Review

After CI passes, please refer to the PR Approval Progress and proactively @ the committers in the table to review the code. After approval, committers will comment /lgtm and /approve. Once the lgtm and approved labels are successfully added, the PR will be merged automatically.

likedislike
CANN-robotCANN-robot成员
11 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
11 天前 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/cannbot-knowledge ✅ qq_41667743, zhanghz1 (2/2) ✅ qq_41667743 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
longjihuilongjihui
11 天前 修改了pull request 的描述
此处折叠了50条消息 查看更多
zhanghz1
zhanghz1成员
7 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
7 天前 添加了label:lgtm
CANN-robot
CANN-robot成员
7 天前 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
汪汪超成员
7 天前 解决了最后一个问题
CANN-robotCANN-robot成员
7 天前 合入了pull request