已合并
[fix] Fix glob pattern to match files in subdirectories #5 #498
xqchen7创建于 6 天前
[fix] Fix glob pattern to match files in subdirectories #5 #498
已合并
xqchen7创建于 6 天前
xqchen7成员
6 天前

Which issue(s) this PR fixes or accomplishes

将 .gitcode/workflows/PR-pipeline_op-plugin.yml 中 Detect_file_changes 的 paths-changes-filter non-doc 排除规则由 !.md、!.rst 等 glob 模式改为 !/*.md、!/*.rst 等,使其能匹配子目录下的文档文件,纯文档变更(含子目录)可正确跳过 Build 阶段。
Fixes #303\

Purpose

Test Plan

构造仅含子目录下 .md/.rst 文档变更的 PR,评论 /compile 触发流水线,验证 need_build=no、Build 阶段被跳过;非文档变更时 need_build=yes、Build 正常执行。

Test Report

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 xqchen7 的贡献)
Xxqchen7成员
6 天前 创建了 pull request,commit 48b7fe0b
atomgit-bot
atomgit-bot
6 天前 评论:

变更摘要

此 PR 修复了 CI 流水线中 paths-changes-filternon-doc 排除规则的 glob 模式缺陷:将 !**.md!**.rst 等模式统一改为 !**/*.md!**/*.rst 等形式。原模式仅能匹配根目录下的文档文件,子目录中的文档文件变更无法被正确识别为非文档变更,导致纯文档 PR 仍会触发 Build 阶段。修复后,所有目录层级下的 .md.rst.log.png.jpg.doc.eddx.svg 文件变更均能被正确排除,纯文档变更可正确跳过 Build。

主要改动

  • 修复 non-doc 排除规则的 glob 模式:在 .gitcode/workflows/PR-pipeline_mindie-sd.yml- stage: detect-file-changes 中,将 8 个 non-doc 排除路径从 !**.md / !**.rst / !**.log / !**.png / !**.jpg / !**.doc / !**.eddx / !**.svg 改为 !**/*.md / !**/*.rst / !**/*.log / !**/*.png / !**/*.jpg / !**/*.doc / !**/*.eddx / !**/*.svg,使排除规则覆盖子目录中的文档文件。
likedislike
atomgit-bot
atomgit-bot
6 天前 评论:

代码审查

✅ 未发现问题

likedislike
Xxiangjie10成员
6 天前 添加了label:pr-audit-failed
xiangjie10成员
6 天前 评论:
🔍 PR 规范审计未通过,以下项目需要修正:
  • PR 未关联里程碑或 Issue

请修正后重新提交,或联系仓库管理员。

likedislike
ascend-robotascend-robot成员
6 天前 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
6 天前 评论:

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/MindIE-SD weixin_44144262, changetheway (2/2) weixin_44144262 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
6 天前 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
6 天前 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
6 天前 添加了label:docs-ci-pipeline-running
Xxqchen7成员
6 天前 修改了pull request 的描述
ascend-robot
ascend-robot成员
6 天前 评论:

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

likedislike
ascend-robotascend-robot成员
6 天前 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6 天前 添加了label:docs-ci-pipeline-success
lijinxi
lijinxi成员
6 天前 评论:

compile

likedislike
Xxqchen7成员
6 天前 update merge request[project id: 8476587, iid: 498, commit_id: 85121febb0987bd8a45bc9bd6501a6c3262cfb1a] virtual merging success
ascend-robot
ascend-robot成员
6 天前 评论:

ascend docs pipeline is running...

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

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

likedislike
ascend-robotascend-robot成员
6 天前 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6 天前 添加了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
6 天前 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
6 天前 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
6 天前 评论:
流水线 PR-pipeline_MindIE-SD_gitcode#1724 [ commitID:03828516 ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build_linux_x86_abi1 >>>
Build_linux_arm_abi1 >>>
Build_linux_arm_abi0 >>>
恶意代码检查 Antipoison >>>
开源片段检查 SCA >>>
开发者测试 UT_linux_arm >>>
流水线 PR-pipeline_MindIE-SD_gitcode >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
Xxqchen7成员
6 天前 删除了label:pr-audit-failed
lijinxi
lijinxi成员
6 天前 评论:

/lgtm

likedislike
lijinxi
lijinxi成员
6 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
6 天前 添加了label:approved
changetheway成员
6 天前 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
6 天前 添加了label:lgtm
lijinxi
lijinxi成员
6 天前 评论:

/merge

likedislike
lanwangli成员
5 天前 评论:

/approve

likedislike
lanwangli成员
5 天前 评论:

/merge

likedislike
ascend-robotascend-robot成员
5 天前 添加了label:keeper_approved
ascend-robotascend-robot成员
5 天前 关闭了关联的issue
ascend-robotascend-robot成员
5 天前 合入了pull request