总体流程与关键概念
在 GitLab 中，代码审查以 合并请求 Merge Request（MR） 为核心：开发者在特性分支完成改动后推送至远端，创建从特性分支到目标分支（如 main/dev）的 MR；审阅者在 MR 页面进行逐行评论、提出改进、批准变更；当满足项目的 审批规则 且 CI 流水线通过 后，由具备权限的维护者完成合并。为提高效率，建议尽早发起 MR、保持 MR 颗粒度适中，并避免直接向受保护分支推送，所有合并应通过 MR 完成。
环境与权限配置
- 安装与初始化（示例）
- 在 CentOS 上可使用官方仓库安装并启动 GitLab（CE/EE），设置 external_url 后执行
gitlab-ctl reconfigure 生效。
- 成员角色与权限
- 常见角色包括 Guest、Reporter、Developer、Maintainer/Owner；建议仅授予资深成员 Maintainer 合并权限，普通开发者使用 Developer 提交 MR。
- 受保护分支
- 在 Settings → Repository → Protected Branches 中配置：对 main/dev 等分支限制直接推送与合并，通常将 Allowed to push 设为 No one，仅允许通过 MR 合并，必要时开启 Required approvals 与 Code Owner approvals。
- 代码规范与前置检查
- 采用一致的 分支命名 与 提交信息规范（如 Angular 规范），并在本地或远端接入 静态检查/代码风格 工具（如 Checkstyle、p3c），以降低审查负担。
标准操作步骤
- 创建特性分支并开发
- 从 dev/main 拉出分支（如
feature/xxx），完成编码与本地自测。
- 提交与推送
git add .、git commit -m "type(scope): subject"、git push origin feature/xxx。
- 创建 MR
- 在项目中点击 Merge Requests → New merge request，选择源/目标分支，填写 标题、描述、标签，指派 Reviewer/Assignee，可设置 Approvals required；描述中建议包含变更背景、影响范围、测试要点与截图/链接。
- 自动化检查
- 推送后自动触发 CI/CD（在
.gitlab-ci.yml 中定义测试、构建、静态扫描等），确保 pipeline 成功 再进入人工审查。
- 进行代码审查
- 审阅者在 Changes 视图逐行评论、提出 Suggestion（可直接在行内应用建议）、对可合并部分点击 Approve；讨论未解决时保持 Open，必要时 Request changes。
- 冲突解决与迭代
- 出现冲突可在 MR 页面使用 Resolve conflicts 工具或本地 rebase/squash 后再次推送；所有反馈处理完再更新 MR。
- 审批与合并
- 达到 Required approvals 且 CI 通过 后，由 Maintainer 执行合并；可选择 Squash and merge 或 Merge commit，并删除源分支以保持仓库整洁。
审查规范与效率提升
- 角色与职责
- 作者 对方案与质量负责，需确保 MR 解决目标问题、无已知缺陷，并附带完成的任务清单与通过 CI；不确定时尽早引入 领域专家/安全团队 参与。
- 审批与范围
- 跨领域改动应获得相应领域 Maintainer/UX/安全 的批准；大型改动可拆分 MR，降低单次审查复杂度。
- 颗粒度与节奏
- 保持 适中颗粒度（一个 MR 聚焦一个功能/修复/改进），尽早创建 Draft/WIP MR 获取早期反馈，减少返工。
- 工具与自动化
- 使用 IDE 插件（如 IntelliJ IDEA 的 GitLab 插件）在编辑器内创建/管理 MR；通过 模板 统一 MR/Issue 信息结构；必要时启用 Duo Chat 辅助解释代码与问题定位。
常见问题与排查
- 无法合并
- 检查 受保护分支 规则、Required approvals 是否达成、CI 流水线 是否全部通过；必要时由 Maintainer 审核并合并。
- 合并冲突
- 在 MR 页面使用 Resolve conflicts 或本地
git rebase main 解决后 git push --force-with-lease；避免直接推送到受保护分支。
- 审批不足或无人审阅
- 在 MR 中明确 指派 Reviewer，并在项目设置中配置 Code Owners 与 审批规则；必要时通过群组/提及提醒相关团队。
- 流水线失败
- 查看 Jobs 日志定位测试/构建/安全扫描失败原因，修复后自动重新运行；确保
.gitlab-ci.yml 配置正确并与代码变更同步更新。