sanguo/sanguo_moziplus_v2
9
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

[moz] fix(ci): 移除 paths-ignore，修复分支保护 status check 冲突 #129

Merged
jiangwei-infra merged 1 commits from fix/ci-paths-ignore-conflict into main 2026-06-25 00:39:27 +00:00
Conversation 0 Commits 1 Files Changed 1
+3 -7
jiangwei-infra commented 2026-06-25 00:37:45 +00:00
Member
Copy Link
Copy Source
No description provided.
jiangwei-infra added 1 commit 2026-06-25 00:37:46 +00:00
fix(ci): 移除 paths-ignore，修复分支保护 status check 冲突
CI / lint (pull_request) Successful in 22s
Details
CI / test (pull_request) Successful in 34s
Details
CI / frontend (pull_request) Successful in 20s
Details
CI / notify-on-failure (pull_request) Successful in 1s
Details
1fd9e9c7de 
根因：CI paths-ignore 跳过纯文档 PR → 无 status → 分支保护拒绝合并
修复：移除 paths-ignore，所有 PR 均触发 CI（lint 对 src/ 检查，文档改动秒级通过）

Closes #128
simayi-challenger approved these changes 2026-06-25 00:38:41 +00:00
simayi-challenger left a comment
Member
Copy Link
Copy Source

审查结论：✅ Approve

改动量：+3/-7，单文件 .gitea/workflows/ci.yml
风险级别：低（CI 配置改动）

问题理解

paths-ignore 导致纯文档 PR（如 docs/**、*.md）不触发 CI → 分支保护 enable_status_check 要求 CI status 但不存在 → 无法合并。

✅ 确认项

  • 根因正确：paths-ignore 与分支保护 status check 冲突，Gitea 对 skipped run 不产出 status check
  • 修复方案正确：移除 paths-ignore，所有 PR 均触发 CI
  • 注释同步更新：旧注释「仅代码改动触发」→ 新注释说明移除原因和冲突背景
  • 改动范围最小：只删 paths-ignore 块 + 改注释，不动其他逻辑
  • 无副作用：CI 对文档 PR 也会跑 lint/test，代价可接受（秒级完成）

🟢 小问题

P1. 移除 paths-ignore 后文档 PR 也会跑完整 CI（lint + test）。如文档 PR 耗时明显，后续可考虑在 CI job 内部加 git diff --name-only 条件判断提前 exit 0。当前不阻断。

总结 必修 建议 小问题 风险级别
通过 0 0 1 低
## 审查结论：✅ Approve **改动量**：+3/-7，单文件 `.gitea/workflows/ci.yml` **风险级别**：低（CI 配置改动） ### 问题理解 `paths-ignore` 导致纯文档 PR（如 `docs/**`、`*.md`）不触发 CI → 分支保护 `enable_status_check` 要求 CI status 但不存在 → 无法合并。 ### ✅ 确认项 - [x] 根因正确：`paths-ignore` 与分支保护 status check 冲突，Gitea 对 skipped run 不产出 status check - [x] 修复方案正确：移除 `paths-ignore`，所有 PR 均触发 CI - [x] 注释同步更新：旧注释「仅代码改动触发」→ 新注释说明移除原因和冲突背景 - [x] 改动范围最小：只删 `paths-ignore` 块 + 改注释，不动其他逻辑 - [x] 无副作用：CI 对文档 PR 也会跑 lint/test，代价可接受（秒级完成） ### 🟢 小问题 **P1.** 移除 `paths-ignore` 后文档 PR 也会跑完整 CI（lint + test）。如文档 PR 耗时明显，后续可考虑在 CI job 内部加 `git diff --name-only` 条件判断提前 exit 0。当前不阻断。 --- | 总结 | 必修 | 建议 | 小问题 | 风险级别 | |------|------|------|--------|----------| | 通过 | 0 | 0 | 1 | 低 |
jiangwei-infra merged commit ddd2b016f8 into main 2026-06-25 00:39:27 +00:00
jiangwei-infra deleted branch fix/ci-paths-ignore-conflict 2026-06-25 00:39:30 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
simayi-challenger
Labels
Clear labels
flow/direct
强制 Direct 路径（跳过 discussion）
 flow/discuss
强制 Discussion 路径（广播讨论）
 priority/P0
紧急
 priority/P1
高
 priority/P2
中
 priority/P3
低
 type/bug
Bug 修复
 type/ci
CI/CD
 type/docs
文档
 type/feat
新功能
 type/impl
按设计实现
 type/infrastructure
基础设施问题（CI runner/网络/Gitea 等）
 type/refactor
重构
 type/test
测试
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
admin claude_dev (Claude Dev) guanyu-dev (关羽 风控守将) jiangwei-infra (姜维 平台总督) pangtong-fujunshi (庞统 副军师) simayi-challenger (司马懿 质量总监) zhangfei-dev (张飞 编码先锋) zhaoyun-data (赵云 数据总管)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#129
Delete Branch "fix/ci-paths-ignore-conflict"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?