已合并
standardization third_party.md #717
duanpengliang创建于 3月9日
standardization third_party.md #717
已合并
duanpengliang创建于 3月9日
duanpengliang
duanpengliang成员
3月9日

描述

standardization third_party.md

关联的Issue

测试

不涉及

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 duanpengliang 的贡献)
duanpengliangduanpengliang成员
3月9日 创建了 pull request,commit 86ca499b
CANN-robot
CANN-robot成员
3月9日 评论:

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/runtime wangtao43, yanmingxiang (2/2) wangtao43 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
3月9日 将wangtao43,zhangpengpeng8,yanmingxiang,Reyn52166,derekxu,tingwood设为评审人
CANN-robotCANN-robot成员
3月9日 将wangtao43,zhangpengpeng8,yanmingxiang,Reyn52166,derekxu,tingwood设为审查人
CANN-robotCANN-robot成员
3月9日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
3月9日 评论:

CLA Signature Pass

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

likedislike
duanpengliangduanpengliang成员
3月9日 修改标题为 “standardization third_party.md”,原标题为“standardization third_party”
duanpengliangduanpengliang成员
3月9日 修改了pull request 的描述
duanpengliangduanpengliang成员
3月9日 修改了pull request 的描述
duanpengliang
duanpengliang成员
3月9日 评论:

compile

likedislike
CANN-robotCANN-robot成员
3月9日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
3月9日 评论:

流水线任务触发成功
任务链接 [4a2e1a4382754edbab500833f6afe279][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
anti_virus ✅ SUCCESS >>>>>
Check_Pr ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_dfx ✅ SUCCESS >>>>>
API_Check ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>

[2026-03-09 20:57:16]    CI执行结束

likedislike
CANN-robotCANN-robot成员
3月9日 添加了label:api-check-pass
CANN-robotCANN-robot成员
3月9日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
3月9日 添加了label:ci-pipeline-passed
zhangjie
zhangjie成员
3月10日 评论:

🤖 CANN 代码审查报告

PR: #717 - standardization third_party.md
严重性: ✅ Low
审查时间: 2026-03-10 02:14


📊 审查结论

✅ 建议合入

  • 严重性: Low
  • 代码质量: 优秀
  • 内存安全: ✅ NA(文档变更)
  • 安全性: ✅ NA(文档变更)
  • 测试覆盖: NA(文档变更)
  • 文档完整性: 完整

纯文档格式化变更,符合 YAML 标准规范,建议合入。


📋 修改概述

本 PR 规范化第三方开源软件清单的 YAML 格式。

  • 修改文件: 1个 (+22行, -22行)
  • 核心变更:
    • Third_Party_Open_Source_Software_List.yaml: 标准化 YAML 缩进格式
      • 统一缩进:3空格 → 4空格(符合 YAML 标准缩进)
      • 类型修正:json, libboundscheck, eigen, openssl, protobuf, abseil-cpp, libseccomp 从 build 改为 run

🔍 代码质量检查

1. 内存安全 ✅

  • 内存泄漏: NA(纯文档变更)
  • 指针操作: NA
  • 动态分配: NA
  • 资源管理: NA

2. 安全性 ✅

  • 输入验证: NA
  • 边界检查: NA
  • 潜在漏洞: NA(无代码变更)

3. 可读性 ✅

  • 代码清晰度: 优秀(YAML 格式标准化)
  • 命名规范: 符合
  • 注释完整性: NA

4. 逻辑正确性 ✅

  • 算法逻辑: NA
  • 边界条件: NA
  • 影响范围: 明确(仅文档格式)

💡 改进建议

无改进建议,文档格式化符合规范。

确认点

  • ✅ YAML 缩进从 3 空格改为 4 空格,符合标准
  • ⚠️ 多个依赖类型从 build 改为 run,请确认这些依赖确实在运行时需要

✅ 代码亮点

  • 格式规范化:统一使用 4 空格缩进,符合 YAML 标准
  • 依赖分类清晰:build/run/test 分类明确
  • CI 通过:已标记 ci-pipeline-passed 和 api-check-pass
  • CLA 已签署:cann-cla/yes
  • 纯文档变更:无代码影响,风险极低

总体评价:文档格式标准化工作,提高了可维护性,建议合入。

likedislike
zhangjie
zhangjie成员
3月10日 评论:

🤖 CANN 代码审查报告

PR: #717 - standardization third_party.md
严重性: ✅ Low
审查时间: 2026-03-10 03:07


📊 审查结论

✅ 建议合入

  • 严重性: Low
  • 代码质量: 优秀
  • 内存安全: ✅ 无风险
  • 安全性: ✅ 无漏洞
  • 测试覆盖: NA
  • 文档完整性: 完整

纯配置文件更新,规范化第三方依赖类型声明,无功能影响。


📋 修改概述

本 PR 对 Third_Party_Open_Source_Software_List.yaml 进行标准化处理:

  1. 格式统一: 将缩进从2空格改为4空格,符合 YAML 规范
  2. 类型修正: 将多个依赖的类型从 build 修正为 run
    • json
    • libboundscheck
    • eigen
    • openssl
    • protobuf
    • abseil-cpp
    • libseccomp
  • 修改文件: 1个 (+22行, -22行)
  • 核心变更: 配置文件格式和类型声明修正

🔍 代码质量检查

1. 内存安全 ✅

  • 内存泄漏: 无风险 - 仅配置文件变更
  • 指针操作: NA
  • 动态分配: NA
  • 资源管理: NA

2. 安全性 ✅

  • 输入验证: NA
  • 边界检查: NA
  • 潜在漏洞: 无 - YAML 格式正确

3. 可读性 ✅

  • 代码清晰度: 优秀 - 格式统一,易于维护
  • 命名规范: 符合 - YAML 规范
  • 注释完整性: NA

4. 逻辑正确性 ✅

  • 算法逻辑: 正确 - 类型声明与实际用途一致
  • 边界条件: NA
  • 影响范围: 明确 - 仅影响依赖声明文档

💡 改进建议

无需额外改进,文档标准化工作完成得很好。


✅ 代码亮点

  1. 格式规范化: 统一缩进为4空格,符合 YAML 最佳实践
  2. 类型准确性: 将实际运行时依赖的库正确标记为 run 类型
  3. 变更清晰: 纯配置变更,无功能影响,易于审查
  4. 维护性提升: 标准化后的配置更易于后续维护和理解

总体评价:规范的文档更新 PR,建议直接合入。✅

likedislike
zhangjie
zhangjie成员
3月10日 评论:

🤖 CANN 代码审查报告

PR: #717 - standardization third_party.md
严重性: ✅ Low
审查时间: 2026-03-10 04:07


📊 审查结论

✅ 建议合入

  • 严重性: Low
  • 代码质量: 良好
  • 内存安全: NA(纯文档)
  • 安全性: NA
  • 测试覆盖: NA(纯文档)
  • 文档完整性: 完整

纯文档更新,标准化第三方软件清单的格式。主要修正缩进对齐,并将部分依赖类型从 build 更新为 run


📋 修改概述

本次 PR 对第三方软件清单进行标准化处理。

  • 修改文件: 1个 (+22行, -22行)
  • 核心变更:
    • Third_Party_Open_Source_Software_List.yaml:
      • 统一缩进格式(4空格)
      • 更新依赖类型:json/libboundscheck/eigen/openssl/protobuf/abseil-cpp/libseccomp 从 build 改为 run

🔍 代码质量检查

1. 内存安全 NA

  • 纯文档变更,不涉及代码

2. 安全性 NA

  • 纯文档变更

3. 可读性 ✅

  • 格式规范: 良好(YAML 格式统一)
  • 缩进对齐: 符合标准

4. 逻辑正确性 ✅

  • 依赖类型: buildrun 的变更需确认是否符合实际运行时依赖
  • 影响范围: 文档级别

💡 改进建议

  1. 确认依赖类型: 多个依赖从 build 改为 run,请确认这些确实是运行时依赖而非编译时依赖

✅ 代码亮点

  • 统一 YAML 格式,提升可读性
  • 文档维护及时

总体评价:纯文档标准化更新,建议合入。

likedislike
ykl999
ykl999成员
3月10日 评论:

/approve

likedislike
王璐瑶成员
3月10日 评论:

/lgtm

likedislike
yanmingxiang成员
3月10日 评论:

/lgtm

likedislike
wangtao成员
3月10日 评论:

/approve

likedislike
CANN-robot
CANN-robot成员
3月10日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
ykl999, wangluyao17

likedislike
CANN-robotCANN-robot成员
3月10日 添加了label:lgtmapproved
CANN-robot
CANN-robot成员
3月10日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: wangtao43.
Reviewers who wrote a comment of /lgtm are: yanmingxiang, wangtao43.

likedislike
CANN-robotCANN-robot成员
3月10日 合入了pull request