Skip to content

docs: 补充CI/CD发布流程说明并修正过时内容 - #379

Merged
AHuangMeow merged 2 commits into
mainfrom
docs/update
Sep 7, 2026
Merged

AHuangMeow merged 2 commits into
mainfrom
docs/update

Conversation

@AHuangMeow

Copy link
Copy Markdown
Contributor

No description provided.

- 新增发布流程章节:合并到 main 自动部署测试环境(preview),promote-* tag 触发生产发布
- 修正服务端口表与架构图:以 Nacos 生产配置为准(gRPC 20000-20010,bff 8080)
- 删除仓库中不存在的脚本说明(build-*.sh/sync-config.sh)
- 添加新服务流程补充 promote.yaml 同步白名单要求
@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

Next included review available in 38 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 06ef6aba-433f-48a8-95fd-48018d2e34e9

📥 Commits

Reviewing files that changed from the base of the PR and between 6a7dbf6 and 0b9c9c6.

📒 Files selected for processing (1)
  • README.md
📝 Walkthrough

Walkthrough

README.md 更新了 11 个服务的端口信息和架构图标注。文档新增 Nacos 按环境管理配置、预发布部署和 promote-* 标签发布流程。新增服务步骤补充了 deploy.yaml 变更检测路径和 promote.yaml 版本同步白名单要求。未发现关键问题。

Merge Risk: 🟡 Moderate · up to 6a7db

该文档更新会指导服务部署和生产发布,但新增服务步骤可能导致服务未构建、未部署或部署失败,且跨午夜执行发布命令可能推送不存在的标签。应先补全说明并复用同一标签值后再合并。

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive 未提供 PR 描述,因此无法确认作者是否说明了文档变更的范围和目的。 补充简要 PR 描述,说明端口更新、Nacos 配置管理、CI/CD 发布流程变更及 promote.yaml 白名单要求。
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 标题准确概括了文档变更的主要内容,包括补充 CI/CD 发布流程和修正过时内容。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/update

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai 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.

Actionable comments posted: 2


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: d4cd3ac4-29fd-4cde-96b2-92506f79b032

📥 Commits

Reviewing files that changed from the base of the PR and between dbc16fe and 6a7dbf6.

📒 Files selected for processing (1)
  • README.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

Comment thread README.md Outdated
Comment thread README.md Outdated
- promote tag 命令改用变量复用同一日期值,避免跨午夜不一致
- 新增服务接入明确四处列表:deploy.yaml paths/all_services/远端白名单及 promote.yaml services
@AHuangMeow
AHuangMeow merged commit a065bad into main Sep 7, 2026
13 checks passed
@AHuangMeow
AHuangMeow deleted the docs/update branch September 7, 2026 09:39
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