ci: add build workflow and PR review skill#430
Merged
lizhimins merged 2 commits intoJul 15, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does
引入 CI 与 PR 评审工具，覆盖前后端编译校验与标准化的 PR 评审流程。
1. GitHub CI (
.github/workflows/ci.yml)在 push / pull_request 到
rocketmq-studio分支时触发，两个并行 job 校验编译：mvn -B clean package -DskipTestsnpm ci && npm run build2. PR 评审 Skill (
.claude/skills/pr-review/SKILL.md)输入 PR 链接即可端到端评审：
gh拉取 PR 与关联 Issue → 本地切分支 → 编译前后端 →docker compose拉起（不改端口 6789/8888）→ 结构化分析总结。内置 PR 标题规范检查（Conventional Commits，依据docs/contributing.md与仓库历史）。Compile check
cd server && mvn -B clean package -DskipTestscd web && npm ci && npm run build