已合并
Fix fmt update error #5296
Jingwei Huang创建于 6月29日
Fix fmt update error #5296
已合并
Jingwei Huang创建于 6月29日
Jingwei Huang
Jingwei Huang成员
6月29日

【合入来源】

如有社区issue,请关联issue链接
请勿携带内部流程信息(需求链接、问题单、内部issue等)

【修改方案】

image.png
三方代码仓损坏,这里新改为完成clone,不阻塞CI流程。后续联系仓库管理员修复。

【资料变更】

不涉及

【接口变更】

不涉及

【功能验证】

不涉及

【CheckList】

不涉及

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 Jingwei Huang 的贡献)
Jingwei HuangJingwei Huang成员
6月29日 创建了 pull request,commit c66495e9
ascend-robotascend-robot成员
6月29日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
6月29日 评论:

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-Ascend/op-plugin renyujin, liangsongwei (2/2) liangsongwei (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
6月29日 添加了label:needs-issue
Jingwei Huang
Jingwei Huang成员
6月29日 评论:

compile

likedislike
ascend-robot
ascend-robot成员
6月29日 评论:

Linking Issue Notice

@huangjingwei , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .

likedislike
ascend-robotascend-robot成员
6月29日 添加了label:ci-pipeline-running
atomgit-bot
atomgit-bot
6月29日 评论:

变更摘要

此 PR 修复了 CI 构建脚本中 git submodule update 的浅克隆错误。原命令使用了 --depth=1 参数进行浅克隆,可能导致子模块更新不完整或失败,移除此参数后改为完整的子模块克隆。

主要改动

  • 移除 --depth=1 浅克隆参数: 在 ci/build.sh 中,将 git submodule update --init --depth=1 --recursive 改为 git submodule update --init --recursive,以确保子模块能够完整拉取全部历史记录,避免因浅克隆导致的更新错误。
likedislike
ascend-robot
ascend-robot成员
6月29日 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
6月29日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
6月29日 评论:

ascend docs pipeline is running...

likedislike
ascend-robot
ascend-robot成员
6月29日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
6月29日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6月29日 添加了label:docs-ci-pipeline-success
ascend-robot
ascend-robot成员
6月29日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
atomgit-bot
atomgit-bot
6月29日 评论:

代码审查


审查总结

已审查文件: ci/build.sh

发现问题:

优先级 数量 说明
P0 0
P1 0
P2 0
P3 1 CI 子模块完整克隆可能导致下载耗时和磁盘占用显著增长

整体风险评估:低。 该变更是为修复 fmt 子模块浅克隆更新失败的有意修复,逻辑正确、无安全风险。唯一的顾虑是 CI 性能/资源方面的影响,取决于 CI runner 的资源配置。建议确认 CI 环境能否承受完整递归子模块克隆的开销。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
Jingwei HuangJingwei Huang成员
6月29日 修改了pull request 的描述
此处折叠了7条事件消息 查看更多
ascend-robotascend-robot成员
6月29日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
6月29日 评论:
流水线 PR-pipeline_op-plugin#2202 [ commitID:7929871e ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build_master_ARM >>>
Build_v2_7_1_ARM >>>
Build_v2_9_0_ARM >>>
Build_v2_10_0_ARM >>>
Build_v2_11_0_ARM >>>
Build_v2_12_0_ARM >>>
Build_master_x86 >>>
Build_v2_7_1_x86 >>>
Build_v2_9_0_x86 >>>
Build_v2_10_0_x86 >>>
Build_v2_11_0_x86 >>>
Build_v2_12_0_x86 >>>
恶意代码检查 Antipoison >>>
编码安全与规范检查 CodeCheck >>>
CodeCheck_pre-commit >>>
开源片段检查 SCA >>>
开发者测试 UT_master_ARM_01 >>>
UT_v2_7_1_ARM >>>
UT_v2_9_0_ARM >>>
UT_v2_10_0_ARM >>>
UT_v2_11_0_ARM >>>
UT_v2_12_0_ARM >>>
流水线 PR-pipeline_op-plugin >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
ascend-robot
ascend-robot成员
6月29日 评论:
流水线 PR-pipeline_op-plugin#2202 [ commitID:7929871e ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build_master_ARM >>>
Build_v2_7_1_ARM >>>
Build_v2_9_0_ARM >>>
Build_v2_10_0_ARM >>>
Build_v2_11_0_ARM >>>
Build_v2_12_0_ARM >>>
Build_master_x86 >>>
Build_v2_7_1_x86 >>>
Build_v2_9_0_x86 >>>
Build_v2_10_0_x86 >>>
Build_v2_11_0_x86 >>>
Build_v2_12_0_x86 >>>
恶意代码检查 Antipoison >>>
编码安全与规范检查 CodeCheck >>>
CodeCheck_pre-commit >>>
开源片段检查 SCA >>>
开发者测试 UT_master_ARM_01 >>>
UT_v2_7_1_ARM >>>
UT_v2_9_0_ARM >>>
UT_v2_10_0_ARM >>>
UT_v2_11_0_ARM >>>
UT_v2_12_0_ARM >>>
流水线 PR-pipeline_op-plugin >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
SSCh_zx成员
6月29日 删除了label:needs-issue
梁松伟
梁松伟成员
6月29日 评论:

/approve

likedislike
ascend-robotascend-robot成员
6月29日 添加了label:approved
renyujin成员
6月29日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
6月29日 添加了label:lgtm
ascend-robotascend-robot成员
6月29日 关闭了关联的issue
ascend-robotascend-robot成员
6月29日 合入了pull request