fix(ci): 修复 tag 触发时 production deploy 被 skipped 的问题 #1625

Merged

xiaoxia merged 2 commits from fix/ci-production-deploy-skip into main 2026-09-02 13:09:23 +08:00

Conversation 3 Commits 2 Files Changed 1

+3 -2

xiaoxia commented 2026-09-02 12:37:39 +08:00

Owner

Copy Link

Copy Source

问题

tag v0.1.131 推送后 CI Run #44774 全部 success，但 Deploy Production 实际被 skipped，生产容器仍是 v0.1.130。

根因

check-frontend-only 只在 PR 事件运行（if: github.event_name == pull_request），tag 触发时被 skipped → frontend-lint 因 needs 依赖被 skipped → build-production needs 中有 skipped job 导致也被 skipped → deploy-production 同理被 skipped。

修复

1. 从 build-production 的 needs 中移除 frontend-lint（前端 lint 不是生产构建的必要条件）
2. if 条件加 !failure() && !cancelled() 确保其他 needs 中被 skipped 的 job 不阻塞生产部署

## 问题 tag v0.1.131 推送后 CI Run #44774 全部 success，但 Deploy Production 实际被 skipped，生产容器仍是 v0.1.130。 ## 根因 `check-frontend-only` 只在 PR 事件运行（`if: github.event_name == pull_request`），tag 触发时被 skipped → `frontend-lint` 因 needs 依赖被 skipped → `build-production` needs 中有 skipped job 导致也被 skipped → `deploy-production` 同理被 skipped。 ## 修复 1. 从 `build-production` 的 needs 中移除 `frontend-lint`（前端 lint 不是生产构建的必要条件） 2. if 条件加 `!failure() && !cancelled()` 确保其他 needs 中被 skipped 的 job 不阻塞生产部署

xiaoxia added 1 commit 2026-09-02 12:37:39 +08:00

fix(ci): 修复 tag 触发时 production deploy 被 skipped 的问题 ... 692d0b3c7e

- 从 build-production needs 移除 frontend-lint（tag事件时被skipped）
- if 条件加 !failure() && !cancelled() 确保 skipped 依赖不阻塞生产部署

gitea-actions bot commented 2026-09-02 12:43:21 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1625
预览链接	https://pr-1625.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1625 | | 预览链接 | [https://pr-1625.preview.xiaoxiajianji.com](https://pr-1625.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

xiaoxia added 1 commit 2026-09-02 12:56:09 +08:00

chore: trigger CI retry 0c636fd913

auto-approve-bot commented 2026-09-02 13:00:51 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1625

⚠️ 问题（1个需要修改）

1. .gitea/workflows/ci-pipeline.yml 第1415行：移除了 frontend-lint 依赖项，可能导致代码质量检查被绕过。
   • 问题描述：该作业从 needs 列表中移除了 frontend-lint，同时新增了 !failure() 判断条件。这意味着当前作业将不再等待 frontend-lint 完成，且其执行状态不再受 frontend-lint 成败的影响。
   • 后果：如果此作业是构建、部署或发布相关的关键步骤，移除该依赖将导致即使前端代码存在 Lint 错误（如潜在的语法隐患或风格问题），流水线仍可能继续执行并发布未通过质量检查的代码。
   • 建议：请确认这是否为故意行为（例如将 Lint 改为非阻塞）。如果 frontend-lint 是质量门禁的一部分，必须将其加回 needs 列表，否则存在发布劣质代码的风险。

💡 建议（0个可选）

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-02 05:00:51 | 模型:}

## 代码审查结果 - PR #1625 ### ⚠️ 问题（1个需要修改） 1. **.gitea/workflows/ci-pipeline.yml 第1415行**：移除了 `frontend-lint` 依赖项，可能导致代码质量检查被绕过。 - **问题描述**：该作业从 `needs` 列表中移除了 `frontend-lint`，同时新增了 `!failure()` 判断条件。这意味着当前作业将不再等待 `frontend-lint` 完成，且其执行状态不再受 `frontend-lint` 成败的影响。 - **后果**：如果此作业是构建、部署或发布相关的关键步骤，移除该依赖将导致即使前端代码存在 Lint 错误（如潜在的语法隐患或风格问题），流水线仍可能继续执行并发布未通过质量检查的代码。 - **建议**：请确认这是否为故意行为（例如将 Lint 改为非阻塞）。如果 `frontend-lint` 是质量门禁的一部分，必须将其加回 `needs` 列表，否则存在发布劣质代码的风险。 ### 💡 建议（0个可选） --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-02 05:00:51 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 278208d31a into main 2026-09-02 13:09:23 +08:00

gitea-actions bot commented 2026-09-02 13:12:52 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1625 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1625 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1625