Skip to content

ci: 修复重检状态权限并接入后端覆盖率 - #29

Merged
yydounai1234 merged 2 commits into
1024XEngineer:mainfrom
pionxe:codex/minimal-ci
Jul 31, 2026
Merged

ci: 修复重检状态权限并接入后端覆盖率#29
yydounai1234 merged 2 commits into
1024XEngineer:mainfrom
pionxe:codex/minimal-ci

Conversation

@pionxe

@pionxe pionxe commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

变更内容

  • CI / required 状态发布移出核心工作流,修复 reusable workflow 权限越界导致的启动失败。
  • 新增后端覆盖率工作流,合并单元测试及 PostgreSQL、Redis 集成测试覆盖率。
  • 增加 70% 覆盖率门槛,并通过 OIDC 上传报告到 Codecov。
  • 在 README 添加测试状态与覆盖率徽章,并更新 CI 文档。

验证

  • Actionlint 检查通过。
  • 186 个后端单元测试通过。
  • 7 个容器集成测试通过。
  • JaCoCo 行覆盖率 73.80%,通过门槛。
  • 前端契约检查与生产构建通过。
  • 前后端镜像、Compose 和 Nginx 校验通过。

已知限制

本次修复解决 PR 27 的工作流权限启动失败;CI Refresh 对陈旧 merge ref 的等待问题不在本次变更范围内。

@pionxe
pionxe requested a review from yydounai1234 July 31, 2026 10:39

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found in this review. I checked the workflow split and status permissions, refresh-run behavior, Codecov/OIDC coverage publication, documentation consistency, and the Maven coverage profile. Targeted validation passed: git diff --check and Maven profile activation/build checks completed successfully.

@yydounai1234 yydounai1234 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yydounai1234
yydounai1234 merged commit 66db5d8 into 1024XEngineer:main Jul 31, 2026
20 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants