团队实践

这一课你将能做到

1. 用 CODEOWNERS 文件实现「谁写的代码谁负责审」的自动化分配
2. 设计团队的 Review SLA 并监控执行（首响时间、合并时长、PR 大小分布）
3. 处理「新人不敢审」「资深审不过来」「跨团队互审」等典型组织问题

为什么要学

Code Review 做到最后是组织问题，不是技术问题。技术工具（CI、Lint、Checklist）解决的是「怎么审」；团队实践解决的是「谁来审、多久审完、审不动怎么办」。

人话：你配置了再好的工具，如果没人知道该找谁审、PR 挂一周没人看、新人不敢点 Approve，review 文化就死了。

先建立图像

团队 Review 健康度仪表盘

分配机制                    时效指标                  参与广度
├─ CODEOWNERS 自动分配     ├─ 首响时间 < 24h         ├─ 全员参与（不只资深）
├─ 负载均衡（轮流）        ├─ 合并周期 < 48h         ├─ 新人也敢审
└─ 紧急通道（hotfix）      ├─ PR 大小中位数 < 300 行  └─ 跨团队互审

          目标状态：
   「任何 PR 提交后 24h 内有人开始审，
    48h 内有结论，作者知道该找谁」

跟着做

步骤 1：配置 CODEOWNERS

.github/CODEOWNERS：

# 全局兜底：任何文件改动都至少通知这两个人
*                                        @zhangsan @lisi

# 后端核心模块：资深后端审
/src/main/java/com/example/course/       @backend-senior
/src/main/java/com/example/payment/      @backend-senior @wangwu

# 数据库迁移：必须 DBA 审
/src/main/resources/db/migration/        @dba-team

# 前端
/web/                                    @frontend-team
/web/components/payment/                 @frontend-team @zhaoliu

# 基础设施：DevOps 审
/Dockerfile                              @devops-team
/k8s/                                    @devops-team
/.github/workflows/                      @devops-team

# 文档：任何人都可以审，但至少一人
*.md                                     @zhangsan @lisi

规则：

• 后面的规则覆盖前面的（越具体越优先）
• 文件被改动时，对应的 owner 自动被加为 reviewer
• 可以配置「必须 owner approve 才能合并」（Branch protection → Require review from Code Owners）

步骤 2：配置 Review 负载均衡

避免「所有 PR 都找同一个资深」：

# .github/workflows/assign-reviewer.yml
name: Auto Assign Reviewer
on:
  pull_request:
    types: [opened]

jobs:
  assign:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/github-script@v7
        with:
          script: |
            // 从 CODEOWNERS 解析候选人，按当前 open PR 数排序，选负载最低的
            const reviewers = ['zhangsan', 'lisi', 'wangwu'];
            const { data: openPRs } = await github.rest.pulls.list({
              owner: context.repo.owner,
              repo: context.repo.repo,
              state: 'open'
            });

            const load = {};
            reviewers.forEach(r => load[r] = 0);
            openPRs.forEach(pr => {
              pr.requested_reviewers.forEach(r => {
                if (load[r.login] !== undefined) load[r.login]++;
              });
            });

            const selected = reviewers.reduce((a, b) =>
              load[a] <= load[b] ? a : b
            );

            await github.rest.pulls.requestReviewers({
              owner: context.repo.owner,
              repo: context.repo.repo,
              pull_number: context.issue.number,
              reviewers: [selected]
            });